diff options
author | David Mitchell <davem@iabyn.com> | 2015-10-10 22:23:19 +0100 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2016-02-03 08:59:40 +0000 |
commit | f7d0774bf175c0069a732fe3cb11e2aa9d9f6956 (patch) | |
tree | 4b39bcf7732c3810d93f5ef7c636ffa5d14f05ac /pp_sys.c | |
parent | 61d3b95a3e3c5316d83f1b0882dcab3dc36addc0 (diff) | |
download | perl-f7d0774bf175c0069a732fe3cb11e2aa9d9f6956.tar.gz |
POPBLOCK: don't set newsp and gimme
This macro used to set these two vars as a side-effect.
Since we now usually access those values before we call POPBLOCK,
it's wasteful to set them again.
Diffstat (limited to 'pp_sys.c')
-rw-r--r-- | pp_sys.c | 5 |
1 files changed, 1 insertions, 4 deletions
@@ -1444,8 +1444,6 @@ PP(pp_leavewrite) IO * const io = GvIOp(gv); PerlIO *ofp; PerlIO *fp; - SV **newsp; - I32 gimme; PERL_CONTEXT *cx; OP *retop; bool is_return = cBOOL(PL_op->op_type == OP_RETURN); @@ -1525,7 +1523,7 @@ PP(pp_leavewrite) POPBLOCK(cx,PL_curpm); retop = cx->blk_sub.retop; POPFORMAT(cx); - SP = newsp; /* ignore retval of formline */ + SP = PL_stack_base + cx->blk_oldsp; /* ignore retval of formline */ if (is_return) /* XXX the semantics of doing 'return' in a format aren't documented. @@ -1556,7 +1554,6 @@ PP(pp_leavewrite) } } PL_formtarget = PL_bodytarget; - PERL_UNUSED_VAR(gimme); RETURNOP(retop); } |