Don’t check IS_PADCONST in pad.c:pad_alloc
authorFather Chrysostomos <sprout@cpan.org>
Wed, 3 Jul 2013 01:29:43 +0000 (18:29 -0700)
committerFather Chrysostomos <sprout@cpan.org>
Fri, 26 Jul 2013 06:48:02 +0000 (23:48 -0700)
Since recent commits have given constants &PL_sv_no names, this check
is redundant, since any slots for constants will have been skipped
over by the sv != &PL_sv_undef check just above.

pad.c

diff --git a/pad.c b/pad.c
index 082dc66..76d8aa4 100644 (file)
--- a/pad.c
+++ b/pad.c
@@ -768,7 +768,7 @@ Perl_pad_alloc(pTHX_ I32 optype, U32 tmptype)
                continue;
            sv = *av_fetch(PL_comppad, PL_padix, TRUE);
            if (!(SvFLAGS(sv) & (SVs_PADTMP | SVs_PADMY)) &&
-               !IS_PADGV(sv) && !IS_PADCONST(sv))
+               !IS_PADGV(sv))
                break;
        }
        if (tmptype & SVf_READONLY) {