Use PERL=../miniperl
[p5sagit/p5-mst-13.2.git] / pp_hot.c
index 1dcec63..13e7c25 100644 (file)
--- a/pp_hot.c
+++ b/pp_hot.c
@@ -90,12 +90,15 @@ PP(pp_and)
 PP(pp_sassign)
 {
     dSP; dPOPTOPssrl;
+    MAGIC *mg;
+
     if (op->op_private & OPpASSIGN_BACKWARDS) {
        SV *temp;
        temp = left; left = right; right = temp;
     }
     if (tainting && tainted && (!SvGMAGICAL(left) || !SvSMAGICAL(left) ||
-                               !mg_find(left, 't'))) {
+                               !((mg = mg_find(left, 't')) && mg->mg_len & 1)))
+    {
        TAINT_NOT;
     }
     SvSetSV(right, left);
@@ -136,25 +139,50 @@ PP(pp_seq)
 
 PP(pp_concat)
 {
-    dSP; dATARGET; dPOPTOPssrl;
+  dSP; dATARGET; tryAMAGICbin(concat,opASSIGN);
+  {
+    dPOPTOPssrl;
     STRLEN len;
     char *s;
+    if (SvGMAGICAL(left))
+        mg_get(left);
     if (TARG != left) {
        s = SvPV(left,len);
        sv_setpvn(TARG,s,len);
     }
+    else if (!SvOK(TARG)) {
+       s = SvPV_force(TARG, len);
+       sv_setpv(TARG, "");     /* Suppress warning. */
+    }
     s = SvPV(right,len);
     sv_catpvn(TARG,s,len);
     SETTARG;
     RETURN;
+  }
 }
 
 PP(pp_padsv)
 {
     dSP; dTARGET;
     XPUSHs(TARG);
-    if (op->op_private & OPpLVAL_INTRO)
-       SAVECLEARSV(curpad[op->op_targ]);
+    if (op->op_flags & OPf_MOD) {
+       if (op->op_private & OPpLVAL_INTRO)
+           SAVECLEARSV(curpad[op->op_targ]);
+        else if (op->op_private & (OPpDEREF_HV|OPpDEREF_AV)) {
+           SV* sv = curpad[op->op_targ];
+            if (SvGMAGICAL(sv))
+                mg_get(sv);
+            if (!SvOK(sv)) {
+                if (SvREADONLY(sv))
+                    croak(no_modify);
+                (void)SvUPGRADE(sv, SVt_RV);
+                SvRV(sv) = (op->op_private & OPpDEREF_HV ?
+                            (SV*)newHV() : (SV*)newAV());
+                SvROK_on(sv);
+                SvSETMAGIC(sv);
+            }
+        }
+    }
     RETURN;
 }
 
@@ -177,7 +205,12 @@ PP(pp_eq)
 PP(pp_preinc)
 {
     dSP;
-    sv_inc(TOPs);
+    if (SvIOK(TOPs)) {
+       ++SvIVX(TOPs);
+       SvFLAGS(TOPs) &= ~(SVf_NOK|SVf_POK|SVp_NOK|SVp_POK);
+    }
+    else
+       sv_inc(TOPs);
     SvSETMAGIC(TOPs);
     return NORMAL;
 }
@@ -243,19 +276,25 @@ PP(pp_print)
     else
        gv = defoutgv;
     if (!(io = GvIO(gv))) {
-       if (dowarn)
-           warn("Filehandle %s never opened", GvNAME(gv));
-       errno = EBADF;
+       if (dowarn) {
+           SV* sv = sv_newmortal();
+            gv_fullname(sv,gv);
+            warn("Filehandle %s never opened", SvPV(sv,na));
+        }
+
+       SETERRNO(EBADF,RMS$_IFI);
        goto just_say_no;
     }
     else if (!(fp = IoOFP(io))) {
        if (dowarn)  {
+           SV* sv = sv_newmortal();
+            gv_fullname(sv,gv);
            if (IoIFP(io))
-               warn("Filehandle %s opened only for input", GvNAME(gv));
+               warn("Filehandle %s opened only for input", SvPV(sv,na));
            else
-               warn("print on closed filehandle %s", GvNAME(gv));
+               warn("print on closed filehandle %s", SvPV(sv,na));
        }
-       errno = EBADF;
+       SETERRNO(EBADF,IoIFP(io)?RMS$_FAC:RMS$_IFI);
        goto just_say_no;
     }
     else {
@@ -330,6 +369,8 @@ PP(pp_rv2av)
        }
        else {
            if (SvTYPE(sv) != SVt_PVGV) {
+               char *sym;
+
                if (SvGMAGICAL(sv)) {
                    mg_get(sv);
                    if (SvROK(sv))
@@ -339,11 +380,14 @@ PP(pp_rv2av)
                    if (op->op_flags & OPf_REF ||
                      op->op_private & HINT_STRICT_REFS)
                        DIE(no_usym, "an ARRAY");
+                   if (GIMME == G_ARRAY)
+                       RETURN;
                    RETPUSHUNDEF;
                }
+               sym = SvPV(sv,na);
                if (op->op_private & HINT_STRICT_REFS)
-                   DIE(no_symref, "an ARRAY");
-               sv = (SV*)gv_fetchpv(SvPV(sv, na), TRUE, SVt_PVAV);
+                   DIE(no_symref, sym, "an ARRAY");
+               sv = (SV*)gv_fetchpv(sym, TRUE, SVt_PVAV);
            }
            av = GvAVn(sv);
            if (op->op_private & OPpLVAL_INTRO)
@@ -398,6 +442,8 @@ PP(pp_rv2hv)
        }
        else {
            if (SvTYPE(sv) != SVt_PVGV) {
+               char *sym;
+
                if (SvGMAGICAL(sv)) {
                    mg_get(sv);
                    if (SvROK(sv))
@@ -407,11 +453,16 @@ PP(pp_rv2hv)
                    if (op->op_flags & OPf_REF ||
                      op->op_private & HINT_STRICT_REFS)
                        DIE(no_usym, "a HASH");
+                   if (GIMME == G_ARRAY) {
+                       SP--;
+                       RETURN;
+                   }
                    RETSETUNDEF;
                }
+               sym = SvPV(sv,na);
                if (op->op_private & HINT_STRICT_REFS)
-                   DIE(no_symref, "a HASH");
-               sv = (SV*)gv_fetchpv(SvPV(sv, na), TRUE, SVt_PVHV);
+                   DIE(no_symref, sym, "a HASH");
+               sv = (SV*)gv_fetchpv(sym, TRUE, SVt_PVHV);
            }
            hv = GvHVn(sv);
            if (op->op_private & OPpLVAL_INTRO)
@@ -477,11 +528,12 @@ PP(pp_aassign)
     ary = Null(AV*);
     hash = Null(HV*);
     while (lelem <= lastlelem) {
+       tainted = 0;            /* Each item stands on its own, taintwise. */
        sv = *lelem++;
        switch (SvTYPE(sv)) {
        case SVt_PVAV:
            ary = (AV*)sv;
-           magic = SvSMAGICAL(ary) != 0;
+           magic = SvMAGICAL(ary) != 0;
            
            av_clear(ary);
            i = 0;
@@ -493,6 +545,7 @@ PP(pp_aassign)
                (void)av_store(ary,i++,sv);
                if (magic)
                    mg_set(sv);
+               tainted = 0;
            }
            break;
        case SVt_PVHV: {
@@ -500,7 +553,7 @@ PP(pp_aassign)
                SV *tmpstr;
 
                hash = (HV*)sv;
-               magic = SvSMAGICAL(hash) != 0;
+               magic = SvMAGICAL(hash) != 0;
                hv_clear(hash);
 
                while (relem < lastrelem) {     /* gobble up all the rest */
@@ -517,6 +570,7 @@ PP(pp_aassign)
                    (void)hv_store(hash,tmps,len,tmpstr,0);
                    if (magic)
                        mg_set(tmpstr);
+                   tainted = 0;
                }
            }
            break;
@@ -546,66 +600,62 @@ PP(pp_aassign)
        if (delaymagic & DM_UID) {
 #ifdef HAS_SETRESUID
            (void)setresuid(uid,euid,(Uid_t)-1);
-#else /* not HAS_SETRESUID */
-#ifdef HAS_SETREUID
+#else
+#  ifdef HAS_SETREUID
            (void)setreuid(uid,euid);
-#else /* not HAS_SETREUID */
-#ifdef HAS_SETRUID
+#  else
+#    ifdef HAS_SETRUID
            if ((delaymagic & DM_UID) == DM_RUID) {
                (void)setruid(uid);
-               delaymagic =~ DM_RUID;
+               delaymagic &= ~DM_RUID;
            }
-#endif /* HAS_SETRUID */
-#endif /* HAS_SETRESUID */
-#ifdef HAS_SETEUID
+#    endif /* HAS_SETRUID */
+#    ifdef HAS_SETEUID
            if ((delaymagic & DM_UID) == DM_EUID) {
                (void)seteuid(uid);
-               delaymagic =~ DM_EUID;
+               delaymagic &= ~DM_EUID;
            }
-#endif /* HAS_SETEUID */
+#    endif /* HAS_SETEUID */
            if (delaymagic & DM_UID) {
                if (uid != euid)
                    DIE("No setreuid available");
                (void)setuid(uid);
            }
-#endif /* not HAS_SETREUID */
+#  endif /* HAS_SETREUID */
+#endif /* HAS_SETRESUID */
            uid = (int)getuid();
            euid = (int)geteuid();
        }
        if (delaymagic & DM_GID) {
 #ifdef HAS_SETRESGID
            (void)setresgid(gid,egid,(Gid_t)-1);
-#else /* not HAS_SETREGID */
-#ifdef HAS_SETREGID
+#else
+#  ifdef HAS_SETREGID
            (void)setregid(gid,egid);
-#else /* not HAS_SETREGID */
-#endif /* not HAS_SETRESGID */
-#ifdef HAS_SETRGID
+#  else
+#    ifdef HAS_SETRGID
            if ((delaymagic & DM_GID) == DM_RGID) {
                (void)setrgid(gid);
-               delaymagic =~ DM_RGID;
+               delaymagic &= ~DM_RGID;
            }
-#endif /* HAS_SETRGID */
-#ifdef HAS_SETRESGID
-           (void)setresgid(gid,egid,(Gid_t)-1);
-#else /* not HAS_SETREGID */
-#ifdef HAS_SETEGID
+#    endif /* HAS_SETRGID */
+#    ifdef HAS_SETEGID
            if ((delaymagic & DM_GID) == DM_EGID) {
                (void)setegid(gid);
-               delaymagic =~ DM_EGID;
+               delaymagic &= ~DM_EGID;
            }
-#endif /* HAS_SETEGID */
+#    endif /* HAS_SETEGID */
            if (delaymagic & DM_GID) {
                if (gid != egid)
                    DIE("No setregid available");
                (void)setgid(gid);
            }
-#endif /* not HAS_SETRESGID */
-#endif /* not HAS_SETREGID */
+#  endif /* HAS_SETREGID */
+#endif /* HAS_SETRESGID */
            gid = (int)getgid();
            egid = (int)getegid();
        }
-       tainting |= (euid != uid || egid != gid);
+       tainting |= (uid && (euid != uid || egid != gid));
     }
     delaymagic = 0;
     if (GIMME == G_ARRAY) {
@@ -616,16 +666,11 @@ PP(pp_aassign)
        RETURN;
     }
     else {
+       dTARGET;
        SP = firstrelem;
-       for (relem = firstrelem; relem <= lastrelem; ++relem) {
-           if (SvOK(*relem)) {
-               dTARGET;
                
-               SETi(lastrelem - firstrelem + 1);
-               RETURN;
-           }
-       }
-       RETSETUNDEF;
+       SETi(lastrelem - firstrelem + 1);
+       RETURN;
     }
 }
 
@@ -642,6 +687,8 @@ PP(pp_match)
     register REGEXP *rx = pm->op_pmregexp;
     I32 gimme = GIMME;
     STRLEN len;
+    I32 minmatch = 0;
+    I32 oldsave = savestack_ix;
 
     if (op->op_flags & OPf_STACKED)
        TARG = POPs;
@@ -669,10 +716,14 @@ PP(pp_match)
        rx->startp[0] = 0;
        if (SvTYPE(TARG) >= SVt_PVMG && SvMAGIC(TARG)) {
            MAGIC* mg = mg_find(TARG, 'g');
-           if (mg && mg->mg_len >= 0)
+           if (mg && mg->mg_len >= 0) {
                rx->endp[0] = rx->startp[0] = s + mg->mg_len; 
+               minmatch = (mg->mg_flags & MGf_MINMATCH);
+           }
        }
     }
+    if (!rx->nparens && !global)
+       gimme = G_SCALAR;                       /* accidental array context? */
     safebase = (gimme == G_ARRAY) || global;
     if (pm->op_pmflags & (PMf_MULTILINE|PMf_SINGLELINE)) {
        SAVEINT(multiline);
@@ -684,6 +735,7 @@ play_it_again:
        t = s = rx->endp[0];
        if (s > strend)
            goto nope;
+       minmatch = (s == rx->startp[0]);
     }
     if (pm->op_pmshort) {
        if (pm->op_pmflags & PMf_SCANFIRST) {
@@ -725,11 +777,7 @@ play_it_again:
            pm->op_pmshort = Nullsv;    /* opt is being useless */
        }
     }
-    if (!rx->nparens && !global) {
-       gimme = G_SCALAR;                       /* accidental array context? */
-       safebase = FALSE;
-    }
-    if (regexec(rx, s, strend, truebase, 0,
+    if (pregexec(rx, s, strend, truebase, minmatch,
       SvSCREAM(TARG) ? TARG : Nullsv,
       safebase)) {
        curpm = pm;
@@ -765,6 +813,7 @@ play_it_again:
                ++rx->endp[0];
            goto play_it_again;
        }
+       LEAVE_SCOPE(oldsave);
        RETURN;
     }
     else {
@@ -776,8 +825,17 @@ play_it_again:
                sv_magic(TARG, (SV*)0, 'g', Nullch, 0);
                mg = mg_find(TARG, 'g');
            }
-           mg->mg_len = rx->startp[0] ? rx->endp[0] - truebase : -1;
+           if (rx->startp[0]) {
+               mg->mg_len = rx->endp[0] - truebase;
+               if (rx->startp[0] == rx->endp[0])
+                   mg->mg_flags |= MGf_MINMATCH;
+               else
+                   mg->mg_flags &= ~MGf_MINMATCH;
+           }
+           else
+               mg->mg_len = -1;
        }
+       LEAVE_SCOPE(oldsave);
        RETPUSHYES;
     }
 
@@ -804,6 +862,7 @@ yup:
        tmps = rx->startp[0] = tmps + (s - t);
        rx->endp[0] = tmps + SvCUR(pm->op_pmshort);
     }
+    LEAVE_SCOPE(oldsave);
     RETPUSHYES;
 
 nope:
@@ -818,6 +877,7 @@ ret_no:
                mg->mg_len = -1;
        }
     }
+    LEAVE_SCOPE(oldsave);
     if (gimme == G_ARRAY)
        RETURN;
     RETPUSHNO;
@@ -894,7 +954,7 @@ do_readline()
                       if (cp[i] == '/') {
                          hasdir = isunix = 1;
                          break;
-               }
+                      }
                       if (cp[i] == ']' || cp[i] == '>' || cp[i] == ':') {
                           hasdir = 1;
                           break;
@@ -921,7 +981,8 @@ do_readline()
                            ok = (fputs(begin,tmpfp) != EOF);
                        }
                        if (cxt) (void)lib$find_file_end(&cxt);
-                       if (ok && sts != RMS$_NMF) ok = 0;
+                       if (ok && sts != RMS$_NMF &&
+                           sts != RMS$_DNF && sts != RMS$_FNF) ok = 0;
                        if (!ok) {
                            fp = NULL;
                        }
@@ -942,7 +1003,7 @@ do_readline()
                sv_setpvn(tmpcmd, cshname, cshlen);
                sv_catpv(tmpcmd, " -cf 'set nonomatch; glob ");
                sv_catsv(tmpcmd, tmpglob);
-               sv_catpv(tmpcmd, "'|");
+               sv_catpv(tmpcmd, "' 2>/dev/null |");
 #else
                sv_setpv(tmpcmd, "echo ");
                sv_catsv(tmpcmd, tmpglob);
@@ -963,7 +1024,7 @@ do_readline()
            SP--;
     }
     if (!fp) {
-       if (dowarn)
+       if (dowarn && io && !(IoFLAGS(io) & IOf_START))
            warn("Read on closed filehandle <%s>", GvENAME(last_in_gv));
        if (GIMME == G_SCALAR) {
            (void)SvOK_off(TARG);
@@ -1168,16 +1229,20 @@ PP(pp_iter)
     dSP;
     register CONTEXT *cx;
     SV *sv;
+    AV* av;
 
     EXTEND(sp, 1);
     cx = &cxstack[cxstack_ix];
     if (cx->cx_type != CXt_LOOP)
        DIE("panic: pp_iter");
+    av = cx->blk_loop.iterary;
+    if (av == stack && cx->blk_loop.iterix >= cx->blk_oldsp)
+       RETPUSHNO;
 
-    if (cx->blk_loop.iterix >= cx->blk_oldsp)
+    if (cx->blk_loop.iterix >= AvFILL(av))
        RETPUSHNO;
 
-    if (sv = AvARRAY(cx->blk_loop.iterary)[++cx->blk_loop.iterix]) {
+    if (sv = AvARRAY(av)[++cx->blk_loop.iterix]) {
        SvTEMP_off(sv);
        *cx->blk_loop.itervar = sv;
     }
@@ -1208,6 +1273,7 @@ PP(pp_subst)
     register REGEXP *rx = pm->op_pmregexp;
     STRLEN len;
     int force_on_match = 0;
+    I32 oldsave = savestack_ix;
 
     if (pm->op_pmflags & PMf_CONST)    /* known replacement string? */
        dstr = POPs;
@@ -1218,7 +1284,7 @@ PP(pp_subst)
        EXTEND(SP,1);
     }
     s = SvPV(TARG, len);
-    if (!SvPOKp(TARG))
+    if (!SvPOKp(TARG) || SvREADONLY(TARG) || (SvTYPE(TARG) == SVt_PVGV))
        force_on_match = 1;
 
   force_it:
@@ -1279,7 +1345,7 @@ PP(pp_subst)
        c = SvPV(dstr, clen);
        if (clen <= rx->minlen) {
                                        /* can do inplace substitution */
-           if (regexec(rx, s, strend, orig, 0,
+           if (pregexec(rx, s, strend, orig, 0,
              SvSCREAM(TARG) ? TARG : Nullsv, safebase)) {
                if (force_on_match) {
                    force_on_match = 0;
@@ -1310,6 +1376,7 @@ PP(pp_subst)
                        (void)SvPOK_only(TARG);
                        SvSETMAGIC(TARG);
                        PUSHs(&sv_yes);
+                       LEAVE_SCOPE(oldsave);
                        RETURN;
                    }
                    /*SUPPRESS 560*/
@@ -1325,6 +1392,7 @@ PP(pp_subst)
                        (void)SvPOK_only(TARG);
                        SvSETMAGIC(TARG);
                        PUSHs(&sv_yes);
+                       LEAVE_SCOPE(oldsave);
                        RETURN;
                    }
                    else if (clen) {
@@ -1334,6 +1402,7 @@ PP(pp_subst)
                        (void)SvPOK_only(TARG);
                        SvSETMAGIC(TARG);
                        PUSHs(&sv_yes);
+                       LEAVE_SCOPE(oldsave);
                        RETURN;
                    }
                    else {
@@ -1341,6 +1410,7 @@ PP(pp_subst)
                        (void)SvPOK_only(TARG);
                        SvSETMAGIC(TARG);
                        PUSHs(&sv_yes);
+                       LEAVE_SCOPE(oldsave);
                        RETURN;
                    }
                    /* NOTREACHED */
@@ -1360,7 +1430,7 @@ PP(pp_subst)
                        d += clen;
                    }
                    s = rx->endp[0];
-               } while (regexec(rx, s, strend, orig, s == m,
+               } while (pregexec(rx, s, strend, orig, s == m,
                    Nullsv, TRUE));     /* (don't match same null twice) */
                if (s != d) {
                    i = strend - s;
@@ -1370,15 +1440,17 @@ PP(pp_subst)
                (void)SvPOK_only(TARG);
                SvSETMAGIC(TARG);
                PUSHs(sv_2mortal(newSViv((I32)iters)));
+               LEAVE_SCOPE(oldsave);
                RETURN;
            }
            PUSHs(&sv_no);
+           LEAVE_SCOPE(oldsave);
            RETURN;
        }
     }
     else
        c = Nullch;
-    if (regexec(rx, s, strend, orig, 0,
+    if (pregexec(rx, s, strend, orig, 0,
       SvSCREAM(TARG) ? TARG : Nullsv, safebase)) {
     long_way:
        if (force_on_match) {
@@ -1411,21 +1483,32 @@ PP(pp_subst)
                sv_catpvn(dstr, c, clen);
            if (once)
                break;
-       } while (regexec(rx, s, strend, orig, s == m, Nullsv,
+       } while (pregexec(rx, s, strend, orig, s == m, Nullsv,
            safebase));
        sv_catpvn(dstr, s, strend - s);
-       sv_replace(TARG, dstr);
+
+       (void)SvOOK_off(TARG);
+       Safefree(SvPVX(TARG));
+       SvPVX(TARG) = SvPVX(dstr);
+       SvCUR_set(TARG, SvCUR(dstr));
+       SvLEN_set(TARG, SvLEN(dstr));
+       SvPVX(dstr) = 0;
+       sv_free(dstr);
+
        (void)SvPOK_only(TARG);
        SvSETMAGIC(TARG);
        PUSHs(sv_2mortal(newSViv((I32)iters)));
+       LEAVE_SCOPE(oldsave);
        RETURN;
     }
     PUSHs(&sv_no);
+    LEAVE_SCOPE(oldsave);
     RETURN;
 
 nope:
     ++BmUSEFUL(pm->op_pmshort);
     PUSHs(&sv_no);
+    LEAVE_SCOPE(oldsave);
     RETURN;
 }
 
@@ -1521,23 +1604,23 @@ PP(pp_entersub)
     HV *stash;
     register CV *cv;
     register CONTEXT *cx;
+    I32 gimme;
 
     if (!sv)
        DIE("Not a CODE reference");
     switch (SvTYPE(sv)) {
     default:
        if (!SvROK(sv)) {
+           char *sym;
+
            if (sv == &sv_yes)          /* unfound import, ignore */
                RETURN;
            if (!SvOK(sv))
                DIE(no_usym, "a subroutine");
+           sym = SvPV(sv,na);
            if (op->op_private & HINT_STRICT_REFS)
-               DIE(no_symref, "a subroutine");
-           gv = gv_fetchpv(SvPV(sv, na), FALSE, SVt_PVCV);
-           if (!gv)
-               cv = 0;
-           else
-               cv = GvCV(gv);
+               DIE(no_symref, sym, "a subroutine");
+           cv = perl_get_cv(sym, TRUE);
            break;
        }
        cv = (CV*)SvRV(sv);
@@ -1565,8 +1648,17 @@ PP(pp_entersub)
 
     if (!CvROOT(cv) && !CvXSUB(cv)) {
        if (gv = CvGV(cv)) {
-           SV *tmpstr = sv_newmortal();
+           SV *tmpstr;
            GV *ngv;
+           if (SvFAKE(cv) && GvCV(gv) != cv) { /* autoloaded stub? */
+               cv = GvCV(gv);
+               if (SvTYPE(sv) == SVt_PVGV) {
+                   SvREFCNT_dec(GvCV((GV*)sv));
+                   GvCV((GV*)sv) = (CV*)SvREFCNT_inc((SV*)cv);
+               }
+               goto retry;
+           }
+           tmpstr = sv_newmortal();
            gv_efullname(tmpstr, gv);
            ngv = gv_fetchmethod(GvESTASH(gv), "AUTOLOAD");
            if (ngv && ngv != gv && (cv = GvCV(ngv))) { /* One more chance... */
@@ -1580,11 +1672,16 @@ PP(pp_entersub)
        DIE("Undefined subroutine called");
     }
 
+    gimme = GIMME;
     if ((op->op_private & OPpDEREF_DB) && !CvXSUB(cv)) {
        sv = GvSV(DBsub);
        save_item(sv);
-       gv = CvGV(cv);
-       gv_efullname(sv,gv);
+       if (SvFLAGS(cv) & (SVpcv_ANON | SVpcv_CLONED)) /* Is GV potentially non-unique? */
+           sv_setsv(sv, newRV((SV*)cv));
+       else {
+           gv = CvGV(cv);
+           gv_efullname(sv,gv);
+       }
        cv = GvCV(DBsub);
        if (!cv)
            DIE("No DBsub routine");
@@ -1607,8 +1704,19 @@ PP(pp_entersub)
            stack_sp = stack_base + items;
        }
        else {
+           I32 markix = TOPMARK;
+
            PUTBACK;
            (void)(*CvXSUB(cv))(cv);
+
+           /* Enforce some sanity in scalar context. */
+           if (gimme == G_SCALAR && ++markix != stack_sp - stack_base ) {
+               if (markix > stack_sp - stack_base)
+                   *(stack_base + markix) = &sv_undef;
+               else
+                   *(stack_base + markix) = *stack_sp;
+               stack_sp = stack_base + markix;
+           }
        }
        LEAVE;
        return NORMAL;
@@ -1617,7 +1725,6 @@ PP(pp_entersub)
        dMARK;
        register I32 items = SP - MARK;
        I32 hasargs = (op->op_flags & OPf_STACKED) != 0;
-       I32 gimme = GIMME;
        AV* padlist = CvPADLIST(cv);
        SV** svp = AvARRAY(padlist);
        push_return(op->op_next);
@@ -1632,21 +1739,28 @@ PP(pp_entersub)
            if (CvDEPTH(cv) > AvFILL(padlist)) {
                AV *av;
                AV *newpad = newAV();
+               SV **oldpad = AvARRAY(svp[CvDEPTH(cv)-1]);
                I32 ix = AvFILL((AV*)svp[1]);
                svp = AvARRAY(svp[0]);
-               while (ix > 0) {
+               for ( ;ix > 0; ix--) {
                    if (svp[ix] != &sv_undef) {
-                       char *name = SvPVX(svp[ix]);    /* XXX */
-                       if (*name == '@')
-                           av_store(newpad, ix--, sv = (SV*)newAV());
-                       else if (*name == '%')
-                           av_store(newpad, ix--, sv = (SV*)newHV());
-                       else
-                           av_store(newpad, ix--, sv = NEWSV(0,0));
-                       SvPADMY_on(sv);
+                       char *name = SvPVX(svp[ix]);
+                       if (SvFLAGS(svp[ix]) & SVf_FAKE) { /* outer lexical? */
+                           av_store(newpad, ix,
+                               SvREFCNT_inc(oldpad[ix]) );
+                       }
+                       else {                          /* our own lexical */
+                           if (*name == '@')
+                               av_store(newpad, ix, sv = (SV*)newAV());
+                           else if (*name == '%')
+                               av_store(newpad, ix, sv = (SV*)newHV());
+                           else
+                               av_store(newpad, ix, sv = NEWSV(0,0));
+                           SvPADMY_on(sv);
+                       }
                    }
                    else {
-                       av_store(newpad, ix--, sv = NEWSV(0,0));
+                       av_store(newpad, ix, sv = NEWSV(0,0));
                        SvPADTMP_on(sv);
                    }
                }
@@ -1704,10 +1818,12 @@ PP(pp_aelem)
 {
     dSP;
     SV** svp;
-    I32 elem = POPi - curcop->cop_arybase;
+    I32 elem = POPi;
     AV *av = (AV*)POPs;
     I32 lval = op->op_flags & OPf_MOD;
 
+    if (elem > 0)
+       elem -= curcop->cop_arybase;
     if (SvTYPE(av) != SVt_PVAV)
        RETPUSHUNDEF;
     svp = av_fetch(av, elem, lval);
@@ -1745,8 +1861,10 @@ PP(pp_method)
     sv = *(stack_base + TOPMARK + 1);
     
     gv = 0;
+    if (SvGMAGICAL(sv))
+        mg_get(sv);
     if (SvROK(sv))
-       ob = SvRV(sv);
+       ob = (SV*)SvRV(sv);
     else {
        GV* iogv;
        char* packname = 0;
@@ -1774,6 +1892,7 @@ DIE("Can't call method \"%s\" without a package or object reference", name);
            SETs(gv);
            RETURN;
        }
+       *(stack_base + TOPMARK + 1) = iogv;
     }
 
     if (!ob || !SvOBJECT(ob)) {