summaryrefslogtreecommitdiff
path: root/pp_sys.c
diff options
context:
space:
mode:
authorDavid Mitchell <davem@iabyn.com>2015-10-10 22:23:19 +0100
committerDavid Mitchell <davem@iabyn.com>2016-02-03 08:59:40 +0000
commitf7d0774bf175c0069a732fe3cb11e2aa9d9f6956 (patch)
tree4b39bcf7732c3810d93f5ef7c636ffa5d14f05ac /pp_sys.c
parent61d3b95a3e3c5316d83f1b0882dcab3dc36addc0 (diff)
downloadperl-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.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/pp_sys.c b/pp_sys.c
index 62287b92f2..d54eb3859b 100644
--- a/pp_sys.c
+++ b/pp_sys.c
@@ -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);
}