diff options
author | Father Chrysostomos <sprout@cpan.org> | 2012-08-26 11:59:19 -0700 |
---|---|---|
committer | Father Chrysostomos <sprout@cpan.org> | 2012-08-26 13:16:54 -0700 |
commit | 9f527363e9a5fddbef57d6741c431af8b53b5470 (patch) | |
tree | 1f6ccc3fd00b64d075c3cd11b7331ee503e17fac /regen/opcode.pl | |
parent | 3da9985538c5eae2dd2de1309b320bc77f757687 (diff) | |
download | perl-9f527363e9a5fddbef57d6741c431af8b53b5470.tar.gz |
pp_hot.c: pp_rv2av: Squash repetitive code
The LVRET that I removed (in the if(SvTYPE(sv) == type) block) actu-
ally never evaluates to true, because that block is only entered for
%hash->{elem} or @array->[elem], in which the parent op is helem or
aelem, not leavesublv or return. LVRET only returns true if the cur-
rent op is the last op in an lvalue sub. Likewise, the OPpMAYBE_LVSUB
flag is never set in that case, so checking it now is harmless (the
cases that used to enter the if(SvTyPE(sv)==type) block now fall
through to the OPpMAYBE_LVSUB check).
(Using LVRET in pp_rv2av is actually incorrect, and I corrected most instances in 40c94d11, but this one remained.)
Diffstat (limited to 'regen/opcode.pl')
0 files changed, 0 insertions, 0 deletions