Mention and discourage use of term 'soft reference'
[p5sagit/p5-mst-13.2.git] / pp_hot.c
index 1dcec63..7f9f513 100644 (file)
--- a/pp_hot.c
+++ b/pp_hot.c
@@ -1,6 +1,6 @@
 /*    pp_hot.c
  *
- *    Copyright (c) 1991-1994, Larry Wall
+ *    Copyright (c) 1991-1997, Larry Wall
  *
  *    You may distribute under the terms of either the GNU General Public
  *    License or the Artistic License, as specified in the README file.
@@ -76,6 +76,68 @@ PP(pp_gv)
     RETURN;
 }
 
+PP(pp_gelem)
+{
+    GV *gv;
+    SV *sv;
+    SV *ref;
+    char *elem;
+    dSP;
+
+    sv = POPs;
+    elem = SvPV(sv, na);
+    gv = (GV*)POPs;
+    ref = Nullsv;
+    sv = Nullsv;
+    switch (elem ? *elem : '\0')
+    {
+    case 'A':
+       if (strEQ(elem, "ARRAY"))
+           ref = (SV*)GvAV(gv);
+       break;
+    case 'C':
+       if (strEQ(elem, "CODE"))
+           ref = (SV*)GvCVu(gv);
+       break;
+    case 'F':
+       if (strEQ(elem, "FILEHANDLE")) /* XXX deprecate in 5.005 */
+           ref = (SV*)GvIOp(gv);
+       break;
+    case 'G':
+       if (strEQ(elem, "GLOB"))
+           ref = (SV*)gv;
+       break;
+    case 'H':
+       if (strEQ(elem, "HASH"))
+           ref = (SV*)GvHV(gv);
+       break;
+    case 'I':
+       if (strEQ(elem, "IO"))
+           ref = (SV*)GvIOp(gv);
+       break;
+    case 'N':
+       if (strEQ(elem, "NAME"))
+           sv = newSVpv(GvNAME(gv), GvNAMELEN(gv));
+       break;
+    case 'P':
+       if (strEQ(elem, "PACKAGE"))
+           sv = newSVpv(HvNAME(GvSTASH(gv)), 0);
+       break;
+    case 'S':
+       if (strEQ(elem, "SCALAR"))
+           ref = GvSV(gv);
+       break;
+    }
+    if (ref)
+       sv = newRV(ref);
+    if (sv)
+       sv_2mortal(sv);
+    else
+       sv = &sv_undef;
+    XPUSHs(sv);
+    RETURN;
+}
+
 PP(pp_and)
 {
     dSP;
@@ -90,14 +152,14 @@ 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'))) {
+    if (tainting && tainted && !SvTAINTED(left))
        TAINT_NOT;
-    }
     SvSetSV(right, left);
     SvSETMAGIC(right);
     SETs(right);
@@ -124,37 +186,43 @@ PP(pp_unstack)
     return NORMAL;
 }
 
-PP(pp_seq)
-{
-    dSP; tryAMAGICbinSET(seq,0); 
-    {
-      dPOPTOPssrl;
-      SETs( sv_eq(left, right) ? &sv_yes : &sv_no );
-      RETURN;
-    }
-}
-
 PP(pp_concat)
 {
-    dSP; dATARGET; dPOPTOPssrl;
+  dSP; dATARGET; tryAMAGICbin(concat,opASSIGN);
+  {
+    dPOPTOPssrl;
     STRLEN len;
     char *s;
     if (TARG != left) {
        s = SvPV(left,len);
        sv_setpvn(TARG,s,len);
     }
+    else if (SvGMAGICAL(TARG))
+       mg_get(TARG);
+    else if (!SvOK(TARG) && SvTYPE(TARG) <= SVt_PVMG) {
+       sv_setpv(TARG, "");     /* Suppress warning. */
+       s = SvPV_force(TARG, len);
+    }
     s = SvPV(right,len);
-    sv_catpvn(TARG,s,len);
+    if (SvOK(TARG))
+       sv_catpvn(TARG,s,len);
+    else
+       sv_setpvn(TARG,s,len);  /* suppress warning */
     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)
+           vivify_ref(curpad[op->op_targ], op->op_flags & OPpDEREF);
+    }
     RETURN;
 }
 
@@ -177,7 +245,16 @@ PP(pp_eq)
 PP(pp_preinc)
 {
     dSP;
-    sv_inc(TOPs);
+    if (SvREADONLY(TOPs) || SvTYPE(TOPs) > SVt_PVLV)
+       croak(no_modify);
+    if (SvIOK(TOPs) && !SvNOK(TOPs) && !SvPOK(TOPs) &&
+       SvIVX(TOPs) != IV_MAX)
+    {
+       ++SvIVX(TOPs);
+       SvFLAGS(TOPs) &= ~(SVp_NOK|SVp_POK);
+    }
+    else
+       sv_inc(TOPs);
     SvSETMAGIC(TOPs);
     return NORMAL;
 }
@@ -197,7 +274,7 @@ PP(pp_add)
 {
     dSP; dATARGET; tryAMAGICbin(add,opASSIGN); 
     {
-      dPOPTOPnnrl;
+      dPOPTOPnnrl_ul;
       SETn( left + right );
       RETURN;
     }
@@ -225,7 +302,19 @@ PP(pp_join)
 PP(pp_pushre)
 {
     dSP;
+#ifdef DEBUGGING
+    /*
+     * We ass_u_me that LvTARGOFF() comes first, and that two STRLENs
+     * will be enough to hold an OP*.
+     */
+    SV* sv = sv_newmortal();
+    sv_upgrade(sv, SVt_PVLV);
+    LvTYPE(sv) = '/';
+    Copy(&op, &LvTARGOFF(sv), 1, OP*);
+    XPUSHs(sv);
+#else
     XPUSHs((SV*)op);
+#endif
     RETURN;
 }
 
@@ -236,26 +325,52 @@ PP(pp_print)
     dSP; dMARK; dORIGMARK;
     GV *gv;
     IO *io;
-    register FILE *fp;
+    register PerlIO *fp;
+    MAGIC *mg;
 
     if (op->op_flags & OPf_STACKED)
        gv = (GV*)*++MARK;
     else
        gv = defoutgv;
+    if (SvMAGICAL(gv) && (mg = mg_find((SV*)gv, 'q'))) {
+       if (MARK == ORIGMARK) {
+           EXTEND(SP, 1);
+           ++MARK;
+           Move(MARK, MARK + 1, (SP - MARK) + 1, SV*);
+           ++SP;
+       }
+       PUSHMARK(MARK - 1);
+       *MARK = mg->mg_obj;
+       PUTBACK;
+       ENTER;
+       perl_call_method("PRINT", G_SCALAR);
+       LEAVE;
+       SPAGAIN;
+       MARK = ORIGMARK + 1;
+       *MARK = *SP;
+       SP = MARK;
+       RETURN;
+    }
     if (!(io = GvIO(gv))) {
-       if (dowarn)
-           warn("Filehandle %s never opened", GvNAME(gv));
-       errno = EBADF;
+       if (dowarn) {
+           SV* sv = sv_newmortal();
+            gv_fullname3(sv, gv, Nullch);
+            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_fullname3(sv, gv, Nullch);
            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 {
@@ -266,7 +381,7 @@ PP(pp_print)
                    break;
                MARK++;
                if (MARK <= SP) {
-                   if (fwrite1(ofs, 1, ofslen, fp) == 0 || ferror(fp)) {
+                   if (PerlIO_write(fp, ofs, ofslen) == 0 || PerlIO_error(fp)) {
                        MARK--;
                        break;
                    }
@@ -284,11 +399,11 @@ PP(pp_print)
            goto just_say_no;
        else {
            if (orslen)
-               if (fwrite1(ors, 1, orslen, fp) == 0 || ferror(fp))
+               if (PerlIO_write(fp, ors, orslen) == 0 || PerlIO_error(fp))
                    goto just_say_no;
 
            if (IoFLAGS(io) & IOf_FLUSH)
-               if (fflush(fp) == EOF)
+               if (PerlIO_flush(fp) == EOF)
                    goto just_say_no;
        }
     }
@@ -305,7 +420,6 @@ PP(pp_print)
 PP(pp_rv2av)
 {
     dSP; dPOPss;
-
     AV *av;
 
     if (SvROK(sv)) {
@@ -329,7 +443,11 @@ PP(pp_rv2av)
            }
        }
        else {
+           GV *gv;
+           
            if (SvTYPE(sv) != SVt_PVGV) {
+               char *sym;
+
                if (SvGMAGICAL(sv)) {
                    mg_get(sv);
                    if (SvROK(sv))
@@ -339,15 +457,22 @@ PP(pp_rv2av)
                    if (op->op_flags & OPf_REF ||
                      op->op_private & HINT_STRICT_REFS)
                        DIE(no_usym, "an ARRAY");
+                   if (dowarn)
+                       warn(warn_uninit);
+                   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");
+               gv = (GV*)gv_fetchpv(sym, TRUE, SVt_PVAV);
+           } else {
+               gv = (GV*)sv;
            }
-           av = GvAVn(sv);
+           av = GvAVn(gv);
            if (op->op_private & OPpLVAL_INTRO)
-               av = save_ary(sv);
+               av = save_ary(gv);
            if (op->op_flags & OPf_REF) {
                PUSHs((SV*)av);
                RETURN;
@@ -371,9 +496,7 @@ PP(pp_rv2av)
 
 PP(pp_rv2hv)
 {
-
     dSP; dTOPss;
-
     HV *hv;
 
     if (SvROK(sv)) {
@@ -397,7 +520,11 @@ PP(pp_rv2hv)
            }
        }
        else {
+           GV *gv;
+           
            if (SvTYPE(sv) != SVt_PVGV) {
+               char *sym;
+
                if (SvGMAGICAL(sv)) {
                    mg_get(sv);
                    if (SvROK(sv))
@@ -407,15 +534,24 @@ PP(pp_rv2hv)
                    if (op->op_flags & OPf_REF ||
                      op->op_private & HINT_STRICT_REFS)
                        DIE(no_usym, "a HASH");
+                   if (dowarn)
+                       warn(warn_uninit);
+                   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");
+               gv = (GV*)gv_fetchpv(sym, TRUE, SVt_PVHV);
+           } else {
+               gv = (GV*)sv;
            }
-           hv = GvHVn(sv);
+           hv = GvHVn(gv);
            if (op->op_private & OPpLVAL_INTRO)
-               hv = save_hash(sv);
+               hv = save_hash(gv);
            if (op->op_flags & OPf_REF) {
                SETs((SV*)hv);
                RETURN;
@@ -430,7 +566,7 @@ PP(pp_rv2hv)
     else {
        dTARGET;
        if (HvFILL(hv)) {
-           sprintf(buf, "%d/%d", HvFILL(hv), HvMAX(hv)+1);
+           sprintf(buf, "%ld/%ld", (long)HvFILL(hv), (long)HvMAX(hv)+1);
            sv_setpv(TARG, buf);
        }
        else
@@ -477,13 +613,15 @@ PP(pp_aassign)
     ary = Null(AV*);
     hash = Null(HV*);
     while (lelem <= lastlelem) {
+       TAINT_NOT;              /* 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);
+           av_extend(ary, lastrelem - relem);
            i = 0;
            while (relem <= lastrelem) {        /* gobble up all the rest */
                sv = NEWSV(28,0);
@@ -493,14 +631,14 @@ PP(pp_aassign)
                (void)av_store(ary,i++,sv);
                if (magic)
                    mg_set(sv);
+               TAINT_NOT;
            }
            break;
        case SVt_PVHV: {
-               char *tmps;
                SV *tmpstr;
 
                hash = (HV*)sv;
-               magic = SvSMAGICAL(hash) != 0;
+               magic = SvMAGICAL(hash) != 0;
                hv_clear(hash);
 
                while (relem < lastrelem) {     /* gobble up all the rest */
@@ -509,15 +647,17 @@ PP(pp_aassign)
                        sv = *(relem++);
                    else
                        sv = &sv_no, relem++;
-                   tmps = SvPV(sv, len);
                    tmpstr = NEWSV(29,0);
                    if (*relem)
                        sv_setsv(tmpstr,*relem);        /* value */
                    *(relem++) = tmpstr;
-                   (void)hv_store(hash,tmps,len,tmpstr,0);
+                   (void)hv_store_ent(hash,sv,tmpstr,0);
                    if (magic)
                        mg_set(tmpstr);
+                   TAINT_NOT;
                }
+               if (relem == lastrelem)
+                   warn("Odd number of elements in hash list");
            }
            break;
        default:
@@ -546,66 +686,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) {
@@ -613,19 +749,17 @@ PP(pp_aassign)
            SP = lastrelem;
        else
            SP = firstrelem + (lastlelem - firstlelem);
+       lelem = firstlelem + (relem - firstrelem);
+       while (relem <= SP)
+           *relem++ = (lelem <= lastlelem) ? *lelem++ : &sv_undef;
        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 +776,9 @@ PP(pp_match)
     register REGEXP *rx = pm->op_pmregexp;
     I32 gimme = GIMME;
     STRLEN len;
+    I32 minmatch = 0;
+    I32 oldsave = savestack_ix;
+    I32 update_minmatch = 1;
 
     if (op->op_flags & OPf_STACKED)
        TARG = POPs;
@@ -653,6 +790,7 @@ PP(pp_match)
     strend = s + len;
     if (!s)
        DIE("panic: do_match");
+    TAINT_NOT;
 
     if (pm->op_pmflags & PMf_USED) {
        if (gimme == G_ARRAY)
@@ -669,11 +807,16 @@ 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);
+               update_minmatch = 0;
+           }
        }
     }
-    safebase = (gimme == G_ARRAY) || global;
+    if (!rx->nparens && !global)
+       gimme = G_SCALAR;                       /* accidental array context? */
+    safebase = (((gimme == G_ARRAY) || global) && !sawampersand);
     if (pm->op_pmflags & (PMf_MULTILINE|PMf_SINGLELINE)) {
        SAVEINT(multiline);
        multiline = pm->op_pmflags & PMf_MULTILINE;
@@ -682,8 +825,10 @@ PP(pp_match)
 play_it_again:
     if (global && rx->startp[0]) {
        t = s = rx->endp[0];
-       if (s > strend)
+       if (s >= strend)
            goto nope;
+       if (update_minmatch++)
+           minmatch = (s == rx->startp[0]);
     }
     if (pm->op_pmshort) {
        if (pm->op_pmflags & PMf_SCANFIRST) {
@@ -710,28 +855,19 @@ play_it_again:
                s = t;
        }
        else if (!multiline) {
-           if (*SvPVX(pm->op_pmshort) != *s ||
-             bcmp(SvPVX(pm->op_pmshort), s, pm->op_pmslen) ) {
-               if (pm->op_pmflags & PMf_FOLD) {
-                   if (ibcmp((U8*)SvPVX(pm->op_pmshort), (U8*)s, pm->op_pmslen) )
-                       goto nope;
-               }
-               else
-                   goto nope;
-           }
+           if (*SvPVX(pm->op_pmshort) != *s
+               || (pm->op_pmslen > 1
+                   && memNE(SvPVX(pm->op_pmshort), s, pm->op_pmslen)))
+               goto nope;
        }
        if (!rx->naughty && --BmUSEFUL(pm->op_pmshort) < 0) {
            SvREFCNT_dec(pm->op_pmshort);
            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,
-      SvSCREAM(TARG) ? TARG : Nullsv,
-      safebase)) {
+    if (pregexec(rx, s, strend, truebase, minmatch,
+                SvSCREAM(TARG) ? TARG : Nullsv, safebase))
+    {
        curpm = pm;
        if (pm->op_pmflags & PMf_ONCE)
            pm->op_pmflags |= PMf_USED;
@@ -742,6 +878,7 @@ play_it_again:
     /*NOTREACHED*/
 
   gotcha:
+    TAINT_IF(rx->exec_tainted);
     if (gimme == G_ARRAY) {
        I32 iters, i, len;
 
@@ -751,6 +888,7 @@ play_it_again:
        else
            i = 0;
        EXTEND(SP, iters + i);
+       EXTEND_MORTAL(iters + i);
        for (i = !i; i <= iters; i++) {
            PUSHs(sv_newmortal());
            /*SUPPRESS 560*/
@@ -761,10 +899,12 @@ play_it_again:
        }
        if (global) {
            truebase = rx->subbeg;
+           strend = rx->subend;
            if (rx->startp[0] && rx->startp[0] == rx->endp[0])
                ++rx->endp[0];
            goto play_it_again;
        }
+       LEAVE_SCOPE(oldsave);
        RETURN;
     }
     else {
@@ -776,16 +916,26 @@ 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] - rx->subbeg;
+               if (rx->startp[0] == rx->endp[0])
+                   mg->mg_flags |= MGf_MINMATCH;
+               else
+                   mg->mg_flags &= ~MGf_MINMATCH;
+           }
        }
+       LEAVE_SCOPE(oldsave);
        RETPUSHYES;
     }
 
 yup:
+    TAINT_IF(rx->exec_tainted);
     ++BmUSEFUL(pm->op_pmshort);
     curpm = pm;
     if (pm->op_pmflags & PMf_ONCE)
        pm->op_pmflags |= PMf_USED;
+    Safefree(rx->subbase);
+    rx->subbase = Nullch;
     if (global) {
        rx->subbeg = truebase;
        rx->subend = strend;
@@ -796,14 +946,13 @@ yup:
     if (sawampersand) {
        char *tmps;
 
-       if (rx->subbase)
-           Safefree(rx->subbase);
        tmps = rx->subbase = savepvn(t, strend-t);
        rx->subbeg = tmps;
        rx->subend = tmps + (strend-t);
        tmps = rx->startp[0] = tmps + (s - t);
        rx->endp[0] = tmps + SvCUR(pm->op_pmshort);
     }
+    LEAVE_SCOPE(oldsave);
     RETPUSHYES;
 
 nope:
@@ -811,13 +960,7 @@ nope:
        ++BmUSEFUL(pm->op_pmshort);
 
 ret_no:
-    if (global) {
-       if (SvTYPE(TARG) >= SVt_PVMG && SvMAGIC(TARG)) {
-           MAGIC* mg = mg_find(TARG, 'g');
-           if (mg)
-               mg->mg_len = -1;
-       }
-    }
+    LEAVE_SCOPE(oldsave);
     if (gimme == G_ARRAY)
        RETURN;
     RETPUSHNO;
@@ -830,10 +973,23 @@ do_readline()
     register SV *sv;
     STRLEN tmplen = 0;
     STRLEN offset;
-    FILE *fp;
+    PerlIO *fp;
     register IO *io = GvIO(last_in_gv);
     register I32 type = op->op_type;
-
+    MAGIC *mg;
+
+    if (SvMAGICAL(last_in_gv) && (mg = mg_find((SV*)last_in_gv, 'q'))) {
+       PUSHMARK(SP);
+       XPUSHs(mg->mg_obj);
+       PUTBACK;
+       ENTER;
+       perl_call_method("READLINE", GIMME);
+       LEAVE;
+       SPAGAIN;
+       if (GIMME == G_SCALAR)
+           SvSetSV_nosteal(TARG, TOPs);
+       RETURN;
+    }
     fp = Nullfp;
     if (io) {
        fp = IoIFP(io);
@@ -870,7 +1026,7 @@ do_readline()
                    char *rstr = rslt + sizeof(unsigned short int), *begin, *end, *cp;
                    char tmpfnam[L_tmpnam] = "SYS$SCRATCH:";
                    $DESCRIPTOR(dfltdsc,"SYS$DISK:[]*.*;");
-                   FILE *tmpfp;
+                   PerlIO *tmpfp;
                    STRLEN i;
                    struct dsc$descriptor_s wilddsc
                       = {0, DSC$K_DTYPE_T, DSC$K_CLASS_S, 0};
@@ -894,13 +1050,13 @@ do_readline()
                       if (cp[i] == '/') {
                          hasdir = isunix = 1;
                          break;
-               }
+                      }
                       if (cp[i] == ']' || cp[i] == '>' || cp[i] == ':') {
                           hasdir = 1;
                           break;
                       }
                    }
-                   if ((tmpfp = fopen(tmpfnam,"w+","fop=dlt")) != NULL) {
+                   if ((tmpfp = PerlIO_open(tmpfnam,"w+","fop=dlt")) != NULL) {
                        ok = ((wilddsc.dsc$a_pointer = tovmsspec(SvPVX(tmpglob),vmsspec)) != NULL);
                        if (ok) wilddsc.dsc$w_length = (unsigned short int) strlen(wilddsc.dsc$a_pointer);
                        while (ok && ((sts = lib$find_file(&wilddsc,&rsdsc,&cxt,
@@ -910,7 +1066,7 @@ do_readline()
                            *(end++) = '\n';  *end = '\0';
                            for (cp = rstr; *cp; cp++) *cp = _tolower(*cp);
                            if (hasdir) {
-                             if (isunix) trim_unixpath(SvPVX(tmpglob),rstr);
+                             if (isunix) trim_unixpath(rstr,SvPVX(tmpglob),1);
                              begin = rstr;
                            }
                            else {
@@ -918,31 +1074,43 @@ do_readline()
                                while (*(--begin) != ']' && *begin != '>') ;
                                ++begin;
                            }
-                           ok = (fputs(begin,tmpfp) != EOF);
+                           ok = (PerlIO_puts(tmpfp,begin) != 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) {
+                           if (!(sts & 1)) {
+                             SETERRNO((sts == RMS$_SYN ? EINVAL : EVMSERR),sts);
+                           }
+                           PerlIO_close(tmpfp);
                            fp = NULL;
                        }
                        else {
-                          rewind(tmpfp);
+                          PerlIO_rewind(tmpfp);
                           IoTYPE(io) = '<';
                           IoIFP(io) = fp = tmpfp;
+                          IoFLAGS(io) &= ~IOf_UNTAINT;  /* maybe redundant */
                        }
                    }
                }
 #else /* !VMS */
 #ifdef DOSISH
+#ifdef OS2
+               sv_setpv(tmpcmd, "for a in ");
+               sv_catsv(tmpcmd, tmpglob);
+               sv_catpv(tmpcmd, "; do echo \"$a\\0\\c\"; done |");
+#else
                sv_setpv(tmpcmd, "perlglob ");
                sv_catsv(tmpcmd, tmpglob);
                sv_catpv(tmpcmd, " |");
-#else
-#ifdef CSH
+#endif /* !OS2 */
+#else /* !DOSISH */
+#if defined(CSH)
                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);
@@ -952,8 +1120,9 @@ do_readline()
                sv_catpv(tmpcmd, "|tr -s ' \t\f\r' '\\n\\n\\n\\n'|");
 #endif
 #endif /* !CSH */
-#endif /* !MSDOS */
-               (void)do_open(last_in_gv, SvPVX(tmpcmd), SvCUR(tmpcmd),Nullfp);
+#endif /* !DOSISH */
+               (void)do_open(last_in_gv, SvPVX(tmpcmd), SvCUR(tmpcmd),
+                             FALSE, 0, 0, Nullfp);
                fp = IoIFP(io);
 #endif /* !VMS */
                LEAVE;
@@ -963,7 +1132,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);
@@ -977,6 +1146,8 @@ do_readline()
     }
     else {
        sv = TARG;
+       if (SvROK(sv))
+           sv_unref(sv);
        (void)SvUPGRADE(sv, SVt_PV);
        tmplen = SvLEN(sv);     /* remember if already alloced */
        if (!tmplen)
@@ -988,7 +1159,7 @@ do_readline()
     }
     for (;;) {
        if (!sv_gets(sv, fp, offset)) {
-           clearerr(fp);
+           PerlIO_clearerr(fp);
            if (IoFLAGS(io) & IOf_ARGV) {
                fp = nextargv(last_in_gv);
                if (fp)
@@ -1005,21 +1176,24 @@ do_readline()
            }
            RETURN;
        }
+       /* This should not be marked tainted if the fp is marked clean */
+       if (!(IoFLAGS(io) & IOf_UNTAINT)) {
+           TAINT;
+           SvTAINTED_on(sv);
+       }
        IoLINES(io)++;
+       SvSETMAGIC(sv);
        XPUSHs(sv);
-       if (tainting) {
-           tainted = TRUE;
-           SvTAINT(sv); /* Anything from the outside world...*/
-       }
        if (type == OP_GLOB) {
            char *tmps;
 
-           if (SvCUR(sv) > 0)
-               SvCUR(sv)--;
-           if (*SvEND(sv) == rschar)
-               *SvEND(sv) = '\0';
-           else
-               SvCUR(sv)++;
+           if (SvCUR(sv) > 0 && SvCUR(rs) > 0) {
+               tmps = SvEND(sv) - 1;
+               if (*tmps == *SvPVX(rs)) {
+                   *tmps = '\0';
+                   SvCUR(sv)--;
+               }
+           }
            for (tmps = SvPVX(sv); *tmps; tmps++)
                if (!isALPHA(*tmps) && !isDIGIT(*tmps) &&
                    strchr("$&*(){}[]'\";\\|?<>~`", *tmps))
@@ -1078,35 +1252,41 @@ PP(pp_enter)
 PP(pp_helem)
 {
     dSP;
-    SV** svp;
+    HE* he;
     SV *keysv = POPs;
-    STRLEN keylen;
-    char *key = SvPV(keysv, keylen);
     HV *hv = (HV*)POPs;
-    I32 lval = op->op_flags & OPf_MOD;
+    U32 lval = op->op_flags & OPf_MOD;
+    U32 defer = op->op_private & OPpLVAL_DEFER;
 
     if (SvTYPE(hv) != SVt_PVHV)
        RETPUSHUNDEF;
-    svp = hv_fetch(hv, key, keylen, lval);
+    he = hv_fetch_ent(hv, keysv, lval && !defer, 0);
     if (lval) {
-       if (!svp || *svp == &sv_undef)
-           DIE(no_helem, key);
-       if (op->op_private & OPpLVAL_INTRO)
-           save_svref(svp);
-       else if (op->op_private & (OPpDEREF_HV|OPpDEREF_AV)) {
-           SV* sv = *svp;
-           if (SvGMAGICAL(sv))
-               mg_get(sv);
-           if (!SvOK(sv)) {
-               (void)SvUPGRADE(sv, SVt_RV);
-               SvRV(sv) = (op->op_private & OPpDEREF_HV ?
-                           (SV*)newHV() : (SV*)newAV());
-               SvROK_on(sv);
-               SvSETMAGIC(sv);
-           }
+       if (!he || HeVAL(he) == &sv_undef) {
+           SV* lv;
+           SV* key2;
+           if (!defer)
+               DIE(no_helem, SvPV(keysv, na));
+           lv = sv_newmortal();
+           sv_upgrade(lv, SVt_PVLV);
+           LvTYPE(lv) = 'y';
+           sv_magic(lv, key2 = newSVsv(keysv), 'y', Nullch, 0);
+           SvREFCNT_dec(key2); /* sv_magic() increments refcount */
+           LvTARG(lv) = SvREFCNT_inc(hv);
+           LvTARGLEN(lv) = 1;
+           PUSHs(lv);
+           RETURN;
+       }
+       if (op->op_private & OPpLVAL_INTRO) {
+           if (HvNAME(hv) && isGV(HeVAL(he)))
+               save_gp((GV*)HeVAL(he), !(op->op_flags & OPf_SPECIAL));
+           else
+               save_svref(&HeVAL(he));
        }
+       else if (op->op_private & OPpDEREF)
+           vivify_ref(HeVAL(he), op->op_private & OPpDEREF);
     }
-    PUSHs(svp ? *svp : &sv_undef);
+    PUSHs(he ? HeVAL(he) : &sv_undef);
     RETURN;
 }
 
@@ -1167,23 +1347,45 @@ PP(pp_iter)
 {
     dSP;
     register CONTEXT *cx;
-    SV *sv;
+    SV* sv;
+    AV* av;
 
     EXTEND(sp, 1);
     cx = &cxstack[cxstack_ix];
     if (cx->cx_type != CXt_LOOP)
        DIE("panic: pp_iter");
 
-    if (cx->blk_loop.iterix >= cx->blk_oldsp)
+    av = cx->blk_loop.iterary;
+    if (cx->blk_loop.iterix >= (av == curstack ? cx->blk_oldsp : AvFILL(av)))
        RETPUSHNO;
 
-    if (sv = AvARRAY(cx->blk_loop.iterary)[++cx->blk_loop.iterix]) {
+    SvREFCNT_dec(*cx->blk_loop.itervar);
+
+    if (sv = AvARRAY(av)[++cx->blk_loop.iterix])
        SvTEMP_off(sv);
-       *cx->blk_loop.itervar = sv;
-    }
     else
-       *cx->blk_loop.itervar = &sv_undef;
+       sv = &sv_undef;
+    if (av != curstack && SvIMMORTAL(sv)) {
+       SV *lv = cx->blk_loop.iterlval;
+       if (lv && SvREFCNT(lv) > 1) {
+           SvREFCNT_dec(lv);
+           lv = Nullsv;
+       }
+       if (lv)
+           SvREFCNT_dec(LvTARG(lv));
+       else {
+           lv = cx->blk_loop.iterlval = NEWSV(26, 0);
+           sv_upgrade(lv, SVt_PVLV);
+           LvTYPE(lv) = 'y';
+           sv_magic(lv, Nullsv, 'y', Nullch, 0);
+       }
+       LvTARG(lv) = SvREFCNT_inc(av);
+       LvTARGOFF(lv) = cx->blk_loop.iterix;
+       LvTARGLEN(lv) = -1;
+       sv = (SV*)lv;
+    }
 
+    *cx->blk_loop.itervar = SvREFCNT_inc(sv);
     RETPUSHYES;
 }
 
@@ -1203,11 +1405,13 @@ PP(pp_subst)
     I32 maxiters;
     register I32 i;
     bool once;
+    bool rxtainted;
     char *orig;
     I32 safebase;
     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;
@@ -1217,9 +1421,14 @@ PP(pp_subst)
        TARG = GvSV(defgv);
        EXTEND(SP,1);
     }
+    if (SvREADONLY(TARG)
+       || (SvTYPE(TARG) > SVt_PVLV
+           && !(SvTYPE(TARG) == SVt_PVGV && SvFAKE(TARG))))
+       croak(no_modify);
     s = SvPV(TARG, len);
-    if (!SvPOKp(TARG))
+    if (!SvPOKp(TARG) || SvTYPE(TARG) == SVt_PVGV)
        force_on_match = 1;
+    TAINT_NOT;
 
   force_it:
     if (!pm || !s)
@@ -1232,7 +1441,7 @@ PP(pp_subst)
        pm = curpm;
        rx = pm->op_pmregexp;
     }
-    safebase = ((!rx || !rx->nparens) && !sawampersand);
+    safebase = (!rx->nparens && !sawampersand);
     if (pm->op_pmflags & (PMf_MULTILINE|PMf_SINGLELINE)) {
        SAVEINT(multiline);
        multiline = pm->op_pmflags & PMf_MULTILINE;
@@ -1259,133 +1468,125 @@ PP(pp_subst)
                s = m;
        }
        else if (!multiline) {
-           if (*SvPVX(pm->op_pmshort) != *s ||
-             bcmp(SvPVX(pm->op_pmshort), s, pm->op_pmslen) ) {
-               if (pm->op_pmflags & PMf_FOLD) {
-                   if (ibcmp((U8*)SvPVX(pm->op_pmshort), (U8*)s, pm->op_pmslen) )
-                       goto nope;
-               }
-               else
-                   goto nope;
-           }
+           if (*SvPVX(pm->op_pmshort) != *s
+               || (pm->op_pmslen > 1
+                   && memNE(SvPVX(pm->op_pmshort), s, pm->op_pmslen)))
+               goto nope;
        }
        if (!rx->naughty && --BmUSEFUL(pm->op_pmshort) < 0) {
            SvREFCNT_dec(pm->op_pmshort);
            pm->op_pmshort = Nullsv;    /* opt is being useless */
        }
     }
+
+    /* only replace once? */
     once = !(rpm->op_pmflags & PMf_GLOBAL);
-    if (rpm->op_pmflags & PMf_CONST) { /* known replacement string? */
-       c = SvPV(dstr, clen);
-       if (clen <= rx->minlen) {
-                                       /* can do inplace substitution */
-           if (regexec(rx, s, strend, orig, 0,
-             SvSCREAM(TARG) ? TARG : Nullsv, safebase)) {
-               if (force_on_match) {
-                   force_on_match = 0;
-                   s = SvPV_force(TARG, len);
-                   goto force_it;
+
+    /* known replacement string? */
+    c = (rpm->op_pmflags & PMf_CONST) ? SvPV(dstr, clen) : Nullch;
+
+    /* can do inplace substitution? */
+    if (c && clen <= rx->minlen) {
+       if (! pregexec(rx, s, strend, orig, 0,
+                      SvSCREAM(TARG) ? TARG : Nullsv, safebase)) {
+           PUSHs(&sv_no);
+           LEAVE_SCOPE(oldsave);
+           RETURN;
+       }
+       if (force_on_match) {
+           force_on_match = 0;
+           s = SvPV_force(TARG, len);
+           goto force_it;
+       }
+       if (rx->subbase)        /* oops, no we can't */
+           goto long_way;
+       d = s;
+       curpm = pm;
+       SvSCREAM_off(TARG);     /* disable possible screamer */
+       if (once) {
+           rxtainted = rx->exec_tainted;
+           m = rx->startp[0];
+           d = rx->endp[0];
+           s = orig;
+           if (m - s > strend - d) {  /* faster to shorten from end */
+               if (clen) {
+                   Copy(c, m, clen, char);
+                   m += clen;
                }
-               if (rx->subbase)        /* oops, no we can't */
-                   goto long_way;
-               d = s;
-               curpm = pm;
-               SvSCREAM_off(TARG);     /* disable possible screamer */
-               if (once) {
-                   m = rx->startp[0];
-                   d = rx->endp[0];
-                   s = orig;
-                   if (m - s > strend - d) {   /* faster to shorten from end */
-                       if (clen) {
-                           Copy(c, m, clen, char);
-                           m += clen;
-                       }
-                       i = strend - d;
-                       if (i > 0) {
-                           Move(d, m, i, char);
-                           m += i;
-                       }
-                       *m = '\0';
-                       SvCUR_set(TARG, m - s);
-                       (void)SvPOK_only(TARG);
-                       SvSETMAGIC(TARG);
-                       PUSHs(&sv_yes);
-                       RETURN;
-                   }
-                   /*SUPPRESS 560*/
-                   else if (i = m - s) {       /* faster from front */
-                       d -= clen;
-                       m = d;
-                       sv_chop(TARG, d-i);
-                       s += i;
-                       while (i--)
-                           *--d = *--s;
-                       if (clen)
-                           Copy(c, m, clen, char);
-                       (void)SvPOK_only(TARG);
-                       SvSETMAGIC(TARG);
-                       PUSHs(&sv_yes);
-                       RETURN;
-                   }
-                   else if (clen) {
-                       d -= clen;
-                       sv_chop(TARG, d);
-                       Copy(c, d, clen, char);
-                       (void)SvPOK_only(TARG);
-                       SvSETMAGIC(TARG);
-                       PUSHs(&sv_yes);
-                       RETURN;
-                   }
-                   else {
-                       sv_chop(TARG, d);
-                       (void)SvPOK_only(TARG);
-                       SvSETMAGIC(TARG);
-                       PUSHs(&sv_yes);
-                       RETURN;
-                   }
-                   /* NOTREACHED */
+               i = strend - d;
+               if (i > 0) {
+                   Move(d, m, i, char);
+                   m += i;
                }
-               do {
-                   if (iters++ > maxiters)
-                       DIE("Substitution loop");
-                   m = rx->startp[0];
-                   /*SUPPRESS 560*/
-                   if (i = m - s) {
-                       if (s != d)
-                           Move(s, d, i, char);
-                       d += i;
-                   }
-                   if (clen) {
-                       Copy(c, d, clen, char);
-                       d += clen;
-                   }
-                   s = rx->endp[0];
-               } while (regexec(rx, s, strend, orig, s == m,
-                   Nullsv, TRUE));     /* (don't match same null twice) */
-               if (s != d) {
-                   i = strend - s;
-                   SvCUR_set(TARG, d - SvPVX(TARG) + i);
-                   Move(s, d, i+1, char);              /* include the Null */
+               *m = '\0';
+               SvCUR_set(TARG, m - s);
+           }
+           /*SUPPRESS 560*/
+           else if (i = m - s) {       /* faster from front */
+               d -= clen;
+               m = d;
+               sv_chop(TARG, d-i);
+               s += i;
+               while (i--)
+                   *--d = *--s;
+               if (clen)
+                   Copy(c, m, clen, char);
+           }
+           else if (clen) {
+               d -= clen;
+               sv_chop(TARG, d);
+               Copy(c, d, clen, char);
+           }
+           else {
+               sv_chop(TARG, d);
+           }
+           TAINT_IF(rxtainted);
+           PUSHs(&sv_yes);
+       }
+       else {
+           rxtainted = 0;
+           do {
+               if (iters++ > maxiters)
+                   DIE("Substitution loop");
+               rxtainted |= rx->exec_tainted;
+               m = rx->startp[0];
+               /*SUPPRESS 560*/
+               if (i = m - s) {
+                   if (s != d)
+                       Move(s, d, i, char);
+                   d += i;
                }
-               (void)SvPOK_only(TARG);
-               SvSETMAGIC(TARG);
-               PUSHs(sv_2mortal(newSViv((I32)iters)));
-               RETURN;
+               if (clen) {
+                   Copy(c, d, clen, char);
+                   d += clen;
+               }
+               s = rx->endp[0];
+           } while (pregexec(rx, s, strend, orig, s == m,
+                             Nullsv, TRUE)); /* don't match same null twice */
+           if (s != d) {
+               i = strend - s;
+               SvCUR_set(TARG, d - SvPVX(TARG) + i);
+               Move(s, d, i+1, char);          /* include the NUL */
            }
-           PUSHs(&sv_no);
-           RETURN;
+           TAINT_IF(rxtainted);
+           PUSHs(sv_2mortal(newSViv((I32)iters)));
        }
+       (void)SvPOK_only(TARG);
+       SvSETMAGIC(TARG);
+       SvTAINT(TARG);
+       LEAVE_SCOPE(oldsave);
+       RETURN;
     }
-    else
-       c = Nullch;
-    if (regexec(rx, s, strend, orig, 0,
-      SvSCREAM(TARG) ? TARG : Nullsv, safebase)) {
+
+    if (pregexec(rx, s, strend, orig, 0,
+                SvSCREAM(TARG) ? TARG : Nullsv, safebase)) {
     long_way:
        if (force_on_match) {
            force_on_match = 0;
            s = SvPV_force(TARG, len);
            goto force_it;
        }
+       rxtainted = rx->exec_tainted;
        dstr = NEWSV(25, sv_len(TARG));
        sv_setpvn(dstr, m, s-m);
        curpm = pm;
@@ -1397,6 +1598,7 @@ PP(pp_subst)
        do {
            if (iters++ > maxiters)
                DIE("Substitution loop");
+           rxtainted |= rx->exec_tainted;
            if (rx->subbase && rx->subbase != orig) {
                m = s;
                s = orig;
@@ -1411,21 +1613,35 @@ PP(pp_subst)
                sv_catpvn(dstr, c, clen);
            if (once)
                break;
-       } while (regexec(rx, s, strend, orig, s == m, Nullsv,
-           safebase));
+       } while (pregexec(rx, s, strend, orig, s == m, Nullsv, safebase));
        sv_catpvn(dstr, s, strend - s);
-       sv_replace(TARG, dstr);
+
+       TAINT_IF(rxtainted);
+
+       (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);
+       SvTAINT(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;
 }
 
@@ -1477,40 +1693,33 @@ PP(pp_leavesub)
     PMOP *newpm;
     I32 gimme;
     register CONTEXT *cx;
+    struct block_sub cxsub;
 
     POPBLOCK(cx,newpm);
-    POPSUB(cx);
-
+    POPSUB1(cx);       /* Delay POPSUB2 until stack values are safe */
     if (gimme == G_SCALAR) {
        MARK = newsp + 1;
        if (MARK <= SP)
-           if (SvFLAGS(TOPs) & SVs_TEMP)
-               *MARK = TOPs;
-           else
-               *MARK = sv_mortalcopy(TOPs);
+           *MARK = SvTEMP(TOPs) ? TOPs : sv_mortalcopy(TOPs);
        else {
-           MEXTEND(mark,0);
+           MEXTEND(MARK, 0);
            *MARK = &sv_undef;
        }
        SP = MARK;
     }
     else {
-       for (mark = newsp + 1; mark <= SP; mark++)
-           if (!(SvFLAGS(*mark) & SVs_TEMP))
-               *mark = sv_mortalcopy(*mark);
-               /* in case LEAVE wipes old return values */
-    }
-
-    if (cx->blk_sub.hasargs) {         /* You don't exist; go away. */
-       AV* av = cx->blk_sub.argarray;
-
-       av_clear(av);
-       AvREAL_off(av);
+       for (MARK = newsp + 1; MARK <= SP; MARK++) {
+           if (!SvTEMP(*MARK))
+               *MARK = sv_mortalcopy(*MARK);
+       }
     }
-    curpm = newpm;     /* Don't pop $1 et al till now */
+    PUTBACK;
+    
+    POPSUB2();         /* Stack values are safe: release CV and @_ ... */
+    curpm = newpm;     /* ... and pop $1 et al */
 
     LEAVE;
-    PUTBACK;
     return pop_return();
 }
 
@@ -1521,23 +1730,29 @@ PP(pp_entersub)
     HV *stash;
     register CV *cv;
     register CONTEXT *cx;
+    I32 gimme;
+    bool hasargs = (op->op_flags & OPf_STACKED) != 0;
 
     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))
+           if (SvGMAGICAL(sv)) {
+               mg_get(sv);
+               sym = SvPOKp(sv) ? SvPVX(sv) : Nullch;
+           }
+           else
+               sym = SvPV(sv, na);
+           if (!sym)
                DIE(no_usym, "a subroutine");
            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);
@@ -1551,7 +1766,7 @@ PP(pp_entersub)
        cv = (CV*)sv;
        break;
     case SVt_PVGV:
-       if (!(cv = GvCV((GV*)sv)))
+       if (!(cv = GvCVu((GV*)sv)))
            cv = sv_2cv(sv, &stash, &gv, TRUE);
        break;
     }
@@ -1564,28 +1779,47 @@ PP(pp_entersub)
        DIE("Not a CODE reference");
 
     if (!CvROOT(cv) && !CvXSUB(cv)) {
-       if (gv = CvGV(cv)) {
-           SV *tmpstr = sv_newmortal();
-           GV *ngv;
-           gv_efullname(tmpstr, gv);
-           ngv = gv_fetchmethod(GvESTASH(gv), "AUTOLOAD");
-           if (ngv && ngv != gv && (cv = GvCV(ngv))) { /* One more chance... */
-               gv = ngv;
-               sv_setsv(GvSV(CvGV(cv)), tmpstr);       /* Set CV's $AUTOLOAD */
-               goto retry;
-           }
-           else
-               DIE("Undefined subroutine &%s called",SvPVX(tmpstr));
+       GV* autogv;
+       SV* subname;
+
+       /* anonymous or undef'd function leaves us no recourse */
+       if (CvANON(cv) || !(gv = CvGV(cv)))
+           DIE("Undefined subroutine called");
+       /* autoloaded stub? */
+       if (cv != GvCV(gv)) {
+           cv = GvCV(gv);
+           goto retry;
        }
-       DIE("Undefined subroutine called");
+       /* should call AUTOLOAD now? */
+       if ((autogv = gv_autoload(GvESTASH(gv), GvNAME(gv), GvNAMELEN(gv)))) {
+           cv = GvCV(autogv);
+           goto retry;
+       }
+       /* sorry */
+       subname = sv_newmortal();
+       gv_efullname3(subname, gv, Nullch);
+       DIE("Undefined subroutine &%s called", SvPVX(subname));
     }
 
-    if ((op->op_private & OPpDEREF_DB) && !CvXSUB(cv)) {
+    gimme = GIMME;
+    if ((op->op_private & OPpENTERSUB_DB) && GvCV(DBsub) && !CvNODEBUG(cv)) {
+       SV *oldsv = sv;
        sv = GvSV(DBsub);
        save_item(sv);
        gv = CvGV(cv);
-       gv_efullname(sv,gv);
+       if ( (CvFLAGS(cv) & (CVf_ANON | CVf_CLONED))
+            || strEQ(GvNAME(gv), "END") 
+            || ((GvCV(gv) != cv) && /* Could be imported, and old sub redefined. */
+                !( (SvTYPE(oldsv) == SVt_PVGV) && (GvCV((GV*)oldsv) == cv)
+                   && (gv = (GV*)oldsv) ))) { /* Use GV from the stack as a fallback. */
+           /* GV is potentially non-unique, or contain different CV. */
+           sv_setsv(sv, newRV((SV*)cv));
+       }
+       else {
+           gv_efullname3(sv, gv, Nullch);
+       }
        cv = GvCV(DBsub);
+       if (CvXSUB(cv)) curcopdb = curcop;
        if (!cv)
            DIE("No DBsub routine");
     }
@@ -1595,6 +1829,7 @@ PP(pp_entersub)
            I32 (*fp3)_((int,int,int));
            dMARK;
            register I32 items = SP - MARK;
+                                       /* We dont worry to copy from @_. */
            while (sp > mark) {
                sp[1] = sp[0];
                sp--;
@@ -1607,8 +1842,43 @@ PP(pp_entersub)
            stack_sp = stack_base + items;
        }
        else {
+           I32 markix = TOPMARK;
+
            PUTBACK;
+
+           if (!hasargs) {
+               /* Need to copy @_ to stack. Alternative may be to
+                * switch stack to @_, and copy return values
+                * back. This would allow popping @_ in XSUB, e.g.. XXXX */
+               AV* av = GvAV(defgv);
+               I32 items = AvFILL(av) + 1;
+
+               if (items) {
+                   /* Mark is at the end of the stack. */
+                   EXTEND(sp, items);
+                   Copy(AvARRAY(av), sp + 1, items, SV*);
+                   sp += items;
+                   PUTBACK ;               
+               }
+           }
+           if (curcopdb) {             /* We assume that the first
+                                          XSUB in &DB::sub is the
+                                          called one. */
+               SAVESPTR(curcop);
+               curcop = curcopdb;
+               curcopdb = NULL;
+           }
+           /* Do we need to open block here? XXXX */
            (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;
@@ -1616,8 +1886,6 @@ PP(pp_entersub)
     else {
        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);
@@ -1627,26 +1895,35 @@ PP(pp_entersub)
        if (CvDEPTH(cv) < 2)
            (void)SvREFCNT_inc(cv);
        else {  /* save temporaries on recursion? */
-           if (CvDEPTH(cv) == 100 && dowarn)
-               warn("Deep recursion on subroutine \"%s\"",GvENAME(CvGV(cv)));
+           if (CvDEPTH(cv) == 100 && dowarn 
+                 && !(perldb && cv == GvCV(DBsub)))
+               sub_crush_depth(cv);
            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? */
+                           || *name == '&')              /* anonymous code? */
+                       {
+                           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);
                    }
                }
@@ -1671,7 +1948,7 @@ PP(pp_entersub)
            }
            cx->blk_sub.savearray = GvAV(defgv);
            cx->blk_sub.argarray = av;
-           GvAV(defgv) = cx->blk_sub.argarray;
+           GvAV(defgv) = (AV*)SvREFCNT_inc(av);
            ++MARK;
 
            if (items > AvMAX(av) + 1) {
@@ -1700,39 +1977,90 @@ PP(pp_entersub)
     }
 }
 
+void
+sub_crush_depth(cv)
+CV* cv;
+{
+    if (CvANON(cv))
+       warn("Deep recursion on anonymous subroutine");
+    else {
+       SV* tmpstr = sv_newmortal();
+       gv_efullname3(tmpstr, CvGV(cv), Nullch);
+       warn("Deep recursion on subroutine \"%s\"", SvPVX(tmpstr));
+    }
+}
+
 PP(pp_aelem)
 {
     dSP;
     SV** svp;
-    I32 elem = POPi - curcop->cop_arybase;
-    AV *av = (AV*)POPs;
-    I32 lval = op->op_flags & OPf_MOD;
+    I32 elem = POPi;
+    AV* av = (AV*)POPs;
+    U32 lval = op->op_flags & OPf_MOD;
+    U32 defer = (op->op_private & OPpLVAL_DEFER) && (elem > AvFILL(av));
 
+    if (elem > 0)
+       elem -= curcop->cop_arybase;
     if (SvTYPE(av) != SVt_PVAV)
        RETPUSHUNDEF;
-    svp = av_fetch(av, elem, lval);
+    svp = av_fetch(av, elem, lval && !defer);
     if (lval) {
-       if (!svp || *svp == &sv_undef)
-           DIE(no_aelem, elem);
+       if (!svp || *svp == &sv_undef) {
+           SV* lv;
+           if (!defer)
+               DIE(no_aelem, elem);
+           lv = sv_newmortal();
+           sv_upgrade(lv, SVt_PVLV);
+           LvTYPE(lv) = 'y';
+           sv_magic(lv, Nullsv, 'y', Nullch, 0);
+           LvTARG(lv) = SvREFCNT_inc(av);
+           LvTARGOFF(lv) = elem;
+           LvTARGLEN(lv) = 1;
+           PUSHs(lv);
+           RETURN;
+       }
        if (op->op_private & OPpLVAL_INTRO)
            save_svref(svp);
-       else if (op->op_private & (OPpDEREF_HV|OPpDEREF_AV)) {
-           SV* sv = *svp;
-           if (SvGMAGICAL(sv))
-               mg_get(sv);
-           if (!SvOK(sv)) {
-               (void)SvUPGRADE(sv, SVt_RV);
-               SvRV(sv) = (op->op_private & OPpDEREF_HV ?
-                           (SV*)newHV() : (SV*)newAV());
-               SvROK_on(sv);
-               SvSETMAGIC(sv);
-           }
-       }
+       else if (op->op_private & OPpDEREF)
+           vivify_ref(*svp, op->op_private & OPpDEREF);
     }
     PUSHs(svp ? *svp : &sv_undef);
     RETURN;
 }
 
+void
+vivify_ref(sv, to_what)
+SV* sv;
+U32 to_what;
+{
+    if (SvGMAGICAL(sv))
+       mg_get(sv);
+    if (!SvOK(sv)) {
+       if (SvREADONLY(sv))
+           croak(no_modify);
+       if (SvTYPE(sv) < SVt_RV)
+           sv_upgrade(sv, SVt_RV);
+       else if (SvTYPE(sv) >= SVt_PV) {
+           (void)SvOOK_off(sv);
+           Safefree(SvPVX(sv));
+           SvLEN(sv) = SvCUR(sv) = 0;
+       }
+       switch (to_what) {
+       case OPpDEREF_SV:
+           SvRV(sv) = newSV(0);
+           break;
+       case OPpDEREF_AV:
+           SvRV(sv) = (SV*)newAV();
+           break;
+       case OPpDEREF_HV:
+           SvRV(sv) = (SV*)newHV();
+           break;
+       }
+       SvROK_on(sv);
+       SvSETMAGIC(sv);
+    }
+}
+
 PP(pp_method)
 {
     dSP;
@@ -1745,14 +2073,17 @@ 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;
+       STRLEN packlen;
 
        if (!SvOK(sv) ||
-           !(packname = SvPV(sv, na)) ||
+           !(packname = SvPV(sv, packlen)) ||
            !(iogv = gv_fetchpv(packname, FALSE, SVt_PVIO)) ||
            !(ob=(SV*)GvIO(iogv)))
        {
@@ -1760,9 +2091,9 @@ PP(pp_method)
            HV *stash;
            if (!packname || !isALPHA(*packname))
 DIE("Can't call method \"%s\" without a package or object reference", name);
-           if (!(stash = gv_stashpv(packname, FALSE))) {
-               if (gv_stashpv("UNIVERSAL", FALSE))
-                   stash = gv_stashpv(packname, TRUE);
+           if (!(stash = gv_stashpvn(packname, packlen, FALSE))) {
+               if (gv_stashpvn("UNIVERSAL", 9, FALSE))
+                   stash = gv_stashpvn(packname, packlen, TRUE);
                else
                    DIE("Can't call method \"%s\" in empty package \"%s\"",
                        name, packname);
@@ -1771,9 +2102,10 @@ DIE("Can't call method \"%s\" without a package or object reference", name);
            if (!gv)
                DIE("Can't locate object method \"%s\" via package \"%s\"",
                    name, packname);
-           SETs(gv);
+           SETs(isGV(gv) ? (SV*)GvCV(gv) : (SV*)gv);
            RETURN;
        }
+       *(stack_base + TOPMARK + 1) = sv_2mortal(newRV((SV*)iogv));
     }
 
     if (!ob || !SvOBJECT(ob)) {
@@ -1789,7 +2121,7 @@ DIE("Can't call method \"%s\" without a package or object reference", name);
                name, HvNAME(SvSTASH(ob)));
     }
 
-    SETs(gv);
+    SETs(isGV(gv) ? (SV*)GvCV(gv) : (SV*)gv);
     RETURN;
 }