diff options
author | David Mitchell <davem@iabyn.com> | 2015-03-05 11:01:01 +0000 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2015-03-05 11:01:01 +0000 |
commit | 010725738074e6477ae945076c10bb2d2042390d (patch) | |
tree | d472449fa416e61589557199fc88e2a9461cc83f /pp.c | |
parent | f93695f990c755bfd70a59f0ac4caf55173fdb4a (diff) | |
download | perl-010725738074e6477ae945076c10bb2d2042390d.tar.gz |
pp.c: remove unneeded SPAGAIN's
Spotted by Coverity.
I've left the SPAGAIN but commented out, in case future changes
to the code make them needed again.
Diffstat (limited to 'pp.c')
-rw-r--r-- | pp.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -5435,7 +5435,7 @@ PP(pp_push) ENTER_with_name("call_PUSH"); call_sv(SV_CONST(PUSH),G_SCALAR|G_DISCARD|G_METHOD_NAMED); LEAVE_with_name("call_PUSH"); - SPAGAIN; + /* SPAGAIN; not needed: SP is assigned to immediately below */ } else { if (SvREADONLY(ary) && MARK < SP) Perl_croak_no_modify(); @@ -5488,7 +5488,7 @@ PP(pp_unshift) ENTER_with_name("call_UNSHIFT"); call_sv(SV_CONST(UNSHIFT),G_SCALAR|G_DISCARD|G_METHOD_NAMED); LEAVE_with_name("call_UNSHIFT"); - SPAGAIN; + /* SPAGAIN; not needed: SP is assigned to immediately below */ } else { SSize_t i = 0; |