]> git.vpit.fr Git - perl/modules/indirect.git/commitdiff
Fix signedness of the pointer<->int conversions
authorVincent Pit <vince@profvince.com>
Thu, 27 Aug 2009 21:48:32 +0000 (23:48 +0200)
committerVincent Pit <vince@profvince.com>
Thu, 27 Aug 2009 22:31:31 +0000 (00:31 +0200)
indirect.xs

index 51199e791badd9c21043fedc4e45095e9a96a731..8f673e12d1cb9bc6a2e7365a3bc35238eae7e0f4 100644 (file)
@@ -305,7 +305,7 @@ STATIC SV *indirect_tag(pTHX_ SV *value) {
   * use the value pointer as the key itself. */
  ptable_hints_store(MY_CXT.tbl, value, h);
 
- return newSVuv(PTR2UV(value));
+ return newSViv(PTR2IV(value));
 }
 
 STATIC SV *indirect_detag(pTHX_ const SV *hint) {
@@ -316,7 +316,7 @@ STATIC SV *indirect_detag(pTHX_ const SV *hint) {
  if (!(hint && SvIOK(hint)))
   return NULL;
 
- h = ptable_fetch(MY_CXT.tbl, INT2PTR(void *, SvUVX(hint)));
+ h = ptable_fetch(MY_CXT.tbl, INT2PTR(void *, SvIVX(hint)));
 
 #if I_WORKAROUND_REQUIRE_PROPAGATION
  {
@@ -349,13 +349,13 @@ STATIC SV *indirect_tag(pTHX_ SV *value) {
  if (SvROK(value)) {
   value = SvRV(value);
   SvREFCNT_inc_simple_NN(value);
-  tag = PTR2UV(value);
+  tag = PTR2IV(value);
  }
 
- return newSVuv(tag);
+ return newSViv(tag);
 }
 
-#define indirect_detag(H) (((H) && SvIOK(H)) ? INT2PTR(SV *, SvUVX(H)) : NULL)
+#define indirect_detag(H) (((H) && SvIOK(H)) ? INT2PTR(SV *, SvIVX(H)) : NULL)
 
 #endif /* I_THREADSAFE || I_WORKAROUND_REQUIRE_PROPAGATION */