This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Re(mov|writ)e two comments from pad.c:pad_alloc
authorFather Chrysostomos <sprout@cpan.org>
Sun, 16 Jun 2013 19:40:05 +0000 (12:40 -0700)
committerFather Chrysostomos <sprout@cpan.org>
Fri, 26 Jul 2013 06:47:59 +0000 (23:47 -0700)
commit833bf1cd69463bf6a806f451560f3a1039bd09ca
tree3fdd0f7b272b13b5457782ecb9c01ec1337c46ea
parentba610af835ade3a2ef1a9e1d4ea4e35aa0bc91a7
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.
pad.c