push in Void Context

The attached patch avoids pushing an IV onto the stack for a push op
in void context.  The "Is there attached magic" branch in the op checks
for void context as does the non-magical branch in the unshift op.

-- c

[Edited version of patch after review by Yitzchak Scott-Thoennes]

diff --git a/pp.c b/pp.c
index e3b7798..fae2d6d 100644 (file)
--- a/pp.c
+++ b/pp.c
@@ -4738,7 +4738,9 @@ PP(pp_push)
 
        PL_delaymagic = 0;
        SP = ORIGMARK;
-       PUSHi( AvFILLp(ary) + 1 );
+       if (OP_GIMME(PL_op, 0) != G_VOID) {
+           PUSHi( AvFILL(ary) + 1 );
+       }
     }
     RETURN;
 }