pp_hot.c:pp_entersub: Rmv comment about setting PL_compcv
authorFather Chrysostomos <sprout@cpan.org>
Sun, 22 Apr 2012 06:25:33 +0000 (23:25 -0700)
committerFather Chrysostomos <sprout@cpan.org>
Mon, 21 May 2012 23:51:28 +0000 (16:51 -0700)
PL_compcv is meant to point to the currently compiling sub, even dur-
ing an eval’s run time.  (See commit 676a678.)  Therefore, this com-
ment’s suggestion is incorrect.

pp_hot.c

index 89165d9..d0cf006 100644 (file)
--- a/pp_hot.c
+++ b/pp_hot.c
@@ -2685,11 +2685,6 @@ try_autoload:
        PUSHSUB(cx);
        cx->blk_sub.retop = PL_op->op_next;
        CvDEPTH(cv)++;
-       /* XXX This would be a natural place to set C<PL_compcv = cv> so
-        * that eval'' ops within this sub know the correct lexical space.
-        * Owing the speed considerations, we choose instead to search for
-        * the cv using find_runcv() when calling doeval().
-        */
        if (CvDEPTH(cv) >= 2) {
            PERL_STACK_OVERFLOW_CHECK();
            pad_push(padlist, CvDEPTH(cv));