summaryrefslogtreecommitdiff
path: root/Cross/warp
diff options
context:
space:
mode:
authorFather Chrysostomos <sprout@cpan.org>2013-06-16 12:40:05 -0700
committerFather Chrysostomos <sprout@cpan.org>2013-07-25 23:47:59 -0700
commit833bf1cd69463bf6a806f451560f3a1039bd09ca (patch)
tree3fdd0f7b272b13b5457782ecb9c01ec1337c46ea /Cross/warp
parentba610af835ade3a2ef1a9e1d4ea4e35aa0bc91a7 (diff)
downloadperl-833bf1cd69463bf6a806f451560f3a1039bd09ca.tar.gz
Re(mov|writ)e two comments from pad.c:pad_alloc
The thing about "foreach" index vars was added in bbce6d697 (insepar- able changes from patch from perl5.003_08 to perl5.003_09, presuma- bly the ‘Lexical scoping cleanup’ part). It is not valid, because ‘foreach’ doesn’t aliases a pad entry to a non-pad (not marked PADMY or PADTMP) value until run time, and pad_alloc happens at compile time. The real reason we need this loop is that entries that close over unavailable variables are not marked PADMY. That may have been a mistake, but it works because of this loop. The reason for the loop also may have changed over time. The comment about copying to sv is not valid, because it is used later on in the same condition when compared to &PL_sv_undef. It was added in commit dd2155a49b.
Diffstat (limited to 'Cross/warp')
0 files changed, 0 insertions, 0 deletions