RE: compile warnings w/ PM_SETRE and ReREFCNT_inc
authorRobin Barker <RMBarker@cpan.org>
Mon, 7 Jan 2008 14:10:30 +0000 (14:10 +0000)
committerRafael Garcia-Suarez <rgarciasuarez@gmail.com>
Wed, 9 Jan 2008 16:32:55 +0000 (16:32 +0000)
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D0939AE@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@32920

pp_ctl.c

index 07f8504..d90d625 100644 (file)
--- a/pp_ctl.c
+++ b/pp_ctl.c
@@ -212,7 +212,8 @@ PP(pp_substcont)
     if(old != rx) {
        if(old)
            ReREFCNT_dec(old);
-       PM_SETRE(pm,ReREFCNT_inc(rx));
+        ReREFCNT_inc(rx);
+       PM_SETRE(pm,rx);
     }
 
     rxres_restore(&cx->sb_rxres, rx);
@@ -309,7 +310,7 @@ PP(pp_substcont)
        mg->mg_len = i;
     }
     if (old != rx)
-       (void)ReREFCNT_inc(rx);
+       ReREFCNT_inc(rx);
     cx->sb_rxtainted |= RX_MATCH_TAINTED(rx);
     rxres_save(&cx->sb_rxres, rx);
     RETURNOP(pm->op_pmstashstartu.op_pmreplstart);
@@ -3828,7 +3829,8 @@ S_make_matcher(pTHX_ REGEXP *re)
 {
     dVAR;
     PMOP *matcher = (PMOP *) newPMOP(OP_MATCH, OPf_WANT_SCALAR | OPf_STACKED);
-    PM_SETRE(matcher, ReREFCNT_inc(re));
+    ReREFCNT_inc(re);
+    PM_SETRE(matcher, re);
     
     SAVEFREEOP((OP *) matcher);
     ENTER; SAVETMPS;