beta3 File::Path warning fix
[p5sagit/p5-mst-13.2.git] / op.c
diff --git a/op.c b/op.c
index b1f867f..6213d24 100644 (file)
--- a/op.c
+++ b/op.c
 #include "EXTERN.h"
 #include "perl.h"
 
+#define USE_OP_MASK  /* Turned on by default in 5.002beta1h */
+
+#ifdef USE_OP_MASK
+/*
+ * In the following definition, the ", (OP *) op" is just to make the compiler
+ * think the expression is of the right type: croak actually does a longjmp.
+ */
+#define CHECKOP(type,op) \
+    ((op_mask && op_mask[type]) \
+     ? (croak("%s trapped by operation mask", op_desc[type]), (OP*)op) \
+     : (*check[type])((OP*)op))
+#else
+#define CHECKOP(type,op) (*check[type])(op)
+#endif /* USE_OP_MASK */
+
 static I32 list_assignment _((OP *op));
-static OP *bad_type _((I32 n, char *t, OP *op, OP *kid));
+static OP *bad_type _((I32 n, char *t, char *name, OP *kid));
 static OP *modkids _((OP *op, I32 type));
 static OP *no_fh_allowed _((OP *op));
 static OP *scalarboolean _((OP *op));
-static OP *too_few_arguments _((OP *op));
-static OP *too_many_arguments _((OP *op));
+static OP *too_few_arguments _((OP *op, char* name));
+static OP *too_many_arguments _((OP *op, char* name));
 static void null _((OP* op));
 static PADOFFSET pad_findlex _((char* name, PADOFFSET newoff, I32 seq,
        CV* startcv, I32 cx_ix));
 
+static char*
+CvNAME(cv)
+CV* cv;
+{
+    SV* tmpsv = sv_newmortal();
+    gv_efullname(tmpsv, CvGV(cv));
+    return SvPV(tmpsv,na);
+}
+
 static OP *
 no_fh_allowed(op)
 OP *op;
 {
     sprintf(tokenbuf,"Missing comma after first argument to %s function",
-       op_name[op->op_type]);
+       op_desc[op->op_type]);
     yyerror(tokenbuf);
     return op;
 }
 
 static OP *
-too_few_arguments(op)
-OP *op;
+too_few_arguments(op, name)
+OP* op;
+char* name;
 {
-    sprintf(tokenbuf,"Not enough arguments for %s", op_name[op->op_type]);
+    sprintf(tokenbuf,"Not enough arguments for %s", name);
     yyerror(tokenbuf);
     return op;
 }
 
 static OP *
-too_many_arguments(op)
+too_many_arguments(op, name)
 OP *op;
+char* name;
 {
-    sprintf(tokenbuf,"Too many arguments for %s", op_name[op->op_type]);
+    sprintf(tokenbuf,"Too many arguments for %s", name);
     yyerror(tokenbuf);
     return op;
 }
 
 static OP *
-bad_type(n, t, op, kid)
+bad_type(n, t, name, kid)
 I32 n;
 char *t;
-OP *op;
+char *name;
 OP *kid;
 {
     sprintf(tokenbuf, "Type of arg %d to %s must be %s (not %s)",
-       (int) n, op_name[op->op_type], t, op_name[kid->op_type]);
+       (int) n, name, t, op_desc[kid->op_type]);
     yyerror(tokenbuf);
     return op;
 }
@@ -76,8 +102,7 @@ OP *op;
 {
     int type = op->op_type;
     if (type != OP_AELEM && type != OP_HELEM) {
-       sprintf(tokenbuf, "Can't use subscript on %s",
-           op_name[type]);
+       sprintf(tokenbuf, "Can't use subscript on %s", op_desc[type]);
        yyerror(tokenbuf);
        if (type == OP_RV2HV || type == OP_ENTERSUB)
            warn("(Did you mean $ or @ instead of %c?)\n",
@@ -118,12 +143,16 @@ char *name;
 }
 
 static PADOFFSET
+#ifndef CAN_PROTOTYPE
 pad_findlex(name, newoff, seq, startcv, cx_ix)
 char *name;
 PADOFFSET newoff;
 I32 seq;
 CV* startcv;
 I32 cx_ix;
+#else
+pad_findlex(char *name, PADOFFSET newoff, I32 seq, CV* startcv, I32 cx_ix)
+#endif
 {
     CV *cv;
     I32 off;
@@ -137,7 +166,7 @@ I32 cx_ix;
        SV** svp = av_fetch(curlist, 0, FALSE);
        AV *curname;
        if (!svp || *svp == &sv_undef)
-           break;
+           continue;
        curname = (AV*)*svp;
        svp = AvARRAY(curname);
        for (off = AvFILL(curname); off > 0; off--) {
@@ -183,7 +212,8 @@ I32 cx_ix;
            }
            break;
        case CXt_EVAL:
-           if (cx->blk_eval.old_op_type != OP_ENTEREVAL)
+           if (cx->blk_eval.old_op_type != OP_ENTEREVAL &&
+               cx->blk_eval.old_op_type != OP_ENTERTRY)
                return 0;       /* require must have its own scope */
            saweval = i;
            break;
@@ -213,7 +243,7 @@ char *name;
     I32 seq = cop_seqmax;
 
     /* The one we're looking for is probably just before comppad_name_fill. */
-    for (off = comppad_name_fill; off > 0; off--) {
+    for (off = AvFILL(comppad_name); off > 0; off--) {
        if ((sv = svp[off]) &&
            sv != &sv_undef &&
            seq <= SvIVX(sv) &&
@@ -406,7 +436,7 @@ OP *op;
        /* FALL THROUGH */
     case OP_PUSHRE:
     case OP_MATCH:
-       regfree(cPMOP->op_pmregexp);
+       pregfree(cPMOP->op_pmregexp);
        SvREFCNT_dec(cPMOP->op_pmshort);
        break;
     default:
@@ -502,6 +532,8 @@ OP *op;
 
     switch (op->op_type) {
     case OP_REPEAT:
+       if (op->op_private & OPpREPEAT_DOLIST)
+           null(((LISTOP*)cBINOP->op_first)->op_first);
        scalar(cBINOP->op_first);
        break;
     case OP_OR:
@@ -574,7 +606,6 @@ OP *op;
     case OP_PADHV:
     case OP_PADANY:
     case OP_AV2ARYLEN:
-    case OP_SV2LEN:
     case OP_REF:
     case OP_REFGEN:
     case OP_SREFGEN:
@@ -638,7 +669,7 @@ OP *op;
     case OP_GGRGID:
     case OP_GETLOGIN:
        if (!(op->op_private & OPpLVAL_INTRO))
-           useless = op_name[op->op_type];
+           useless = op_desc[op->op_type];
        break;
 
     case OP_RV2GV:
@@ -684,7 +715,7 @@ OP *op;
 
     case OP_REPEAT:
        scalarvoid(cBINOP->op_first);
-       useless = op_name[op->op_type];
+       useless = op_desc[op->op_type];
        break;
 
     case OP_OR:
@@ -882,8 +913,8 @@ I32 type;
        if (type == OP_GREPSTART || type == OP_ENTERSUB || type == OP_REFGEN)
            break;
        sprintf(tokenbuf, "Can't modify %s in %s",
-           op_name[op->op_type],
-           type ? op_name[type] : "local");
+           op_desc[op->op_type],
+           type ? op_desc[type] : "local");
        yyerror(tokenbuf);
        return op;
 
@@ -937,6 +968,8 @@ I32 type;
        modcount = 10000;
        break;
     case OP_RV2SV:
+       if (!type && cUNOP->op_first->op_type != OP_GV)
+           croak("Can't localize a reference");
        ref(cUNOP->op_first, op->op_type); 
        /* FALL THROUGH */
     case OP_UNDEF:
@@ -977,7 +1010,7 @@ I32 type;
        sv_magic(sv, Nullsv, mtype, Nullch, 0);
        curpad[op->op_targ] = sv;
        if (op->op_flags & OPf_KIDS)
-           mod(cBINOP->op_first, type);
+           mod(cBINOP->op_first->op_sibling, type);
        break;
 
     case OP_AELEM:
@@ -1050,6 +1083,7 @@ I32 type;
            op->op_ppaddr = ppaddr[OP_RV2CV];
            assert(cUNOP->op_first->op_type == OP_NULL);
            null(((LISTOP*)cUNOP->op_first)->op_first); /* disable pushmark */
+           op->op_flags |= OPf_SPECIAL;
        }
        break;
       
@@ -1059,6 +1093,8 @@ I32 type;
        break;
     case OP_RV2SV:
        ref(cUNOP->op_first, op->op_type);
+       /* FALL THROUGH */
+    case OP_PADSV:
        if (type == OP_RV2AV || type == OP_RV2HV) {
            op->op_private |= (type == OP_RV2AV ? OPpDEREF_AV : OPpDEREF_HV);
            op->op_flags |= OPf_MOD;
@@ -1128,7 +1164,7 @@ OP *op;
             type != OP_PADHV &&
             type != OP_PUSHMARK)
     {
-       sprintf(tokenbuf, "Can't declare %s in my", op_name[op->op_type]);
+       sprintf(tokenbuf, "Can't declare %s in my", op_desc[op->op_type]);
        yyerror(tokenbuf);
        return op;
     }
@@ -1188,7 +1224,7 @@ scope(o)
 OP *o;
 {
     if (o) {
-       if (o->op_flags & OPf_PARENS || perldb) {
+       if (o->op_flags & OPf_PARENS || perldb || tainting) {
            o = prepend_elem(OP_LINESEQ, newOP(OP_ENTER, 0), o);
            o->op_type = OP_LEAVE;
            o->op_ppaddr = ppaddr[OP_LEAVE];
@@ -1369,17 +1405,28 @@ register OP *o;
        return o;
 
     if (!(hints & HINT_INTEGER)) {
+       int vars = 0;
+
        if (type == OP_DIVIDE || !(o->op_flags & OPf_KIDS))
            return o;
 
        for (curop = ((UNOP*)o)->op_first; curop; curop = curop->op_sibling) {
            if (curop->op_type == OP_CONST) {
-               if (SvIOK(((SVOP*)curop)->op_sv))
+               if (SvIOK(((SVOP*)curop)->op_sv)) {
+                   if (SvIVX(((SVOP*)curop)->op_sv) < 0 && vars++)
+                       return o;       /* negatives truncate wrong way, alas */
                    continue;
+               }
                return o;
            }
            if (opargs[curop->op_type] & OA_RETINTEGER)
                continue;
+           if (curop->op_type == OP_PADSV || curop->op_type == OP_RV2SV) {
+               if (vars++)
+                   return o;
+               if (o->op_type >= OP_LT && o->op_type <= OP_NCMP)
+                   continue;   /* allow $i < 10000 to integerize */
+           }
            return o;
        }
        o->op_ppaddr = ppaddr[++(o->op_type)];
@@ -1437,7 +1484,7 @@ OP* op;
     op->op_ppaddr = ppaddr[type];
     op->op_flags |= flags;
 
-    op = (*check[type])(op);
+    op = CHECKOP(type, op);
     if (op->op_type != type)
        return op;
 
@@ -1608,12 +1655,12 @@ I32 flags;
     op->op_flags = flags;
 
     op->op_next = op;
-    /* op->op_private = 0; */
+    op->op_private = 0 + (flags >> 8);
     if (opargs[type] & OA_RETSCALAR)
        scalar(op);
     if (opargs[type] & OA_TARGET)
        op->op_targ = pad_alloc(type, SVs_PADTMP);
-    return (*check[type])(op);
+    return CHECKOP(type, op);
 }
 
 OP *
@@ -1634,9 +1681,9 @@ OP* first;
     unop->op_ppaddr = ppaddr[type];
     unop->op_first = first;
     unop->op_flags = flags | OPf_KIDS;
-    unop->op_private = 1;
+    unop->op_private = 1 | (flags >> 8);
 
-    unop = (UNOP*)(*check[type])((OP*)unop);
+    unop = (UNOP*) CHECKOP(type, unop);
     if (unop->op_next)
        return (OP*)unop;
 
@@ -1662,14 +1709,14 @@ OP* last;
     binop->op_flags = flags | OPf_KIDS;
     if (!last) {
        last = first;
-       binop->op_private = 1;
+       binop->op_private = 1 | (flags >> 8);
     }
     else {
-       binop->op_private = 2;
+       binop->op_private = 2 | (flags >> 8);
        first->op_sibling = last;
     }
 
-    binop = (BINOP*)(*check[type])((OP*)binop);
+    binop = (BINOP*)CHECKOP(type, binop);
     if (binop->op_next)
        return (OP*)binop;
 
@@ -1688,8 +1735,8 @@ OP *repl;
     SV *rstr = ((SVOP*)repl)->op_sv;
     STRLEN tlen;
     STRLEN rlen;
-    register char *t = SvPV(tstr, tlen);
-    register char *r = SvPV(rstr, rlen);
+    register U8 *t = (U8*)SvPV(tstr, tlen);
+    register U8 *r = (U8*)SvPV(rstr, rlen);
     register I32 i;
     register I32 j;
     I32 delete;
@@ -1704,19 +1751,19 @@ OP *repl;
     if (complement) {
        Zero(tbl, 256, short);
        for (i = 0; i < tlen; i++)
-           tbl[t[i] & 0377] = -1;
+           tbl[t[i]] = -1;
        for (i = 0, j = 0; i < 256; i++) {
            if (!tbl[i]) {
                if (j >= rlen) {
                    if (delete)
                        tbl[i] = -2;
                    else if (rlen)
-                       tbl[i] = r[j-1] & 0377;
+                       tbl[i] = r[j-1];
                    else
                        tbl[i] = i;
                }
                else
-                   tbl[i] = r[j++] & 0377;
+                   tbl[i] = r[j++];
            }
        }
     }
@@ -1729,14 +1776,14 @@ OP *repl;
        for (i = 0, j = 0; i < tlen; i++,j++) {
            if (j >= rlen) {
                if (delete) {
-                   if (tbl[t[i] & 0377] == -1)
-                       tbl[t[i] & 0377] = -2;
+                   if (tbl[t[i]] == -1)
+                       tbl[t[i]] = -2;
                    continue;
                }
                --j;
            }
-           if (tbl[t[i] & 0377] == -1)
-               tbl[t[i] & 0377] = r[j] & 0377;
+           if (tbl[t[i]] == -1)
+               tbl[t[i]] = r[j];
        }
     }
     op_free(expr);
@@ -1756,7 +1803,7 @@ I32 flags;
     pmop->op_type = type;
     pmop->op_ppaddr = ppaddr[type];
     pmop->op_flags = flags;
-    pmop->op_private = 0;
+    pmop->op_private = 0 | (flags >> 8);
 
     /* link into pm list */
     if (type != OP_TRANS && curstash) {
@@ -1790,7 +1837,7 @@ OP *repl;
            p = SvPV(pat, plen);
            pm->op_pmflags |= PMf_SKIPWHITE;
        }
-       pm->op_pmregexp = regcomp(p, p + plen, pm);
+       pm->op_pmregexp = pregcomp(p, p + plen, pm);
        if (strEQ("\\s+", pm->op_pmregexp->precomp)) 
            pm->op_pmflags |= PMf_WHITE;
        hoistmust(pm);
@@ -1860,6 +1907,7 @@ OP *repl;
        }
        if (curop == repl) {
            pm->op_pmflags |= PMf_CONST;        /* const for long enough */
+           pm->op_pmpermflags |= PMf_CONST;    /* const for long enough */
            prepend_elem(op->op_type, scalar(repl), op);
        }
        else {
@@ -1901,7 +1949,7 @@ SV *sv;
        scalar((OP*)svop);
     if (opargs[type] & OA_TARGET)
        svop->op_targ = pad_alloc(type, SVs_PADTMP);
-    return (*check[type])((OP*)svop);
+    return CHECKOP(type, svop);
 }
 
 OP *
@@ -1921,7 +1969,7 @@ GV *gv;
        scalar((OP*)gvop);
     if (opargs[type] & OA_TARGET)
        gvop->op_targ = pad_alloc(type, SVs_PADTMP);
-    return (*check[type])((OP*)gvop);
+    return CHECKOP(type, gvop);
 }
 
 OP *
@@ -1941,29 +1989,7 @@ char *pv;
        scalar((OP*)pvop);
     if (opargs[type] & OA_TARGET)
        pvop->op_targ = pad_alloc(type, SVs_PADTMP);
-    return (*check[type])((OP*)pvop);
-}
-
-OP *
-newCVOP(type, flags, cv, cont)
-I32 type;
-I32 flags;
-CV *cv;
-OP *cont;
-{
-    CVOP *cvop;
-    Newz(1101, cvop, 1, CVOP);
-    cvop->op_type = type;
-    cvop->op_ppaddr = ppaddr[type];
-    cvop->op_cv = cv;
-    cvop->op_cont = cont;
-    cvop->op_next = (OP*)cvop;
-    cvop->op_flags = flags;
-    if (opargs[type] & OA_RETSCALAR)
-       scalar((OP*)cvop);
-    if (opargs[type] & OA_TARGET)
-       cvop->op_targ = pad_alloc(type, SVs_PADTMP);
-    return (*check[type])((OP*)cvop);
+    return CHECKOP(type, pvop);
 }
 
 void
@@ -1992,8 +2018,9 @@ OP *op;
 }
 
 void
-utilize(aver, id, arg)
+utilize(aver, floor, id, arg)
 int aver;
+I32 floor;
 OP *id;
 OP *arg;
 {
@@ -2005,27 +2032,31 @@ OP *arg;
     if (id->op_type != OP_CONST)
        croak("Module name must be constant");
 
-    meth = newSVOP(OP_CONST, 0,
-       aver
-           ? newSVpv("import", 6)
-           : newSVpv("unimport", 8)
-       );
-
-    /* Make copy of id so we don't free it twice */
-    pack = newSVOP(OP_CONST, 0, newSVsv(((SVOP*)id)->op_sv));
-
-    /* Fake up a require */
-    rqop = newUNOP(OP_REQUIRE, 0, id);
-
     /* Fake up an import/unimport */
-    imop = convert(OP_ENTERSUB, OPf_STACKED|OPf_SPECIAL,
+    if (arg && arg->op_type == OP_STUB)
+       imop = arg;             /* no import on explicit () */
+    else {
+       /* Make copy of id so we don't free it twice */
+       pack = newSVOP(OP_CONST, 0, newSVsv(((SVOP*)id)->op_sv));
+
+       meth = newSVOP(OP_CONST, 0,
+           aver
+               ? newSVpv("import", 6)
+               : newSVpv("unimport", 8)
+           );
+       imop = convert(OP_ENTERSUB, OPf_STACKED|OPf_SPECIAL,
                    append_elem(OP_LIST,
                        prepend_elem(OP_LIST, pack, list(arg)),
                        newUNOP(OP_METHOD, 0, meth)));
+    }
+
+    /* Fake up a require */
+    rqop = newUNOP(OP_REQUIRE, 0, id);
 
     /* Fake up the BEGIN {}, which does its thing immediately. */
-    newSUB(start_subparse(),
+    newSUB(floor,
        newSVOP(OP_CONST, 0, newSVpv("BEGIN", 5)),
+       Nullop,
        append_elem(OP_LINESEQ,
            newSTATEOP(0, Nullch, rqop),
            newSTATEOP(0, Nullch, imop) ));
@@ -2112,33 +2143,10 @@ OP *right;
            op_free(right);
            return Nullop;
        }
-       if (right && right->op_type == OP_SPLIT) {
-           if ((op = ((LISTOP*)right)->op_first) && op->op_type == OP_PUSHRE) {
-               PMOP *pm = (PMOP*)op;
-               if (left->op_type == OP_RV2AV &&
-                   !(left->op_private & OPpLVAL_INTRO) )
-               {
-                   op = ((UNOP*)left)->op_first;
-                   if (op->op_type == OP_GV && !pm->op_pmreplroot) {
-                       pm->op_pmreplroot = (OP*)((GVOP*)op)->op_gv;
-                       pm->op_pmflags |= PMf_ONCE;
-                       op_free(left);
-                       return right;
-                   }
-               }
-               else {
-                   if (modcount < 10000) {
-                       SV *sv = ((SVOP*)((LISTOP*)right)->op_last)->op_sv;
-                       if (SvIVX(sv) == 0)
-                           sv_setiv(sv, modcount+1);
-                   }
-               }
-           }
-       }
        op = newBINOP(OP_AASSIGN, flags,
                list(force_list(right)),
                list(force_list(left)) );
-       op->op_private = 0;
+       op->op_private = 0 | (flags >> 8);
        if (!(left->op_private & OPpLVAL_INTRO)) {
            static int generation = 100;
            OP *curop;
@@ -2157,7 +2165,7 @@ OP *right;
                             curop->op_type == OP_PADHV ||
                             curop->op_type == OP_PADANY) {
                        SV **svp = AvARRAY(comppad_name);
-                       SV *sv = svp[curop->op_targ];;
+                       SV *sv = svp[curop->op_targ];
                        if (SvCUR(sv) == generation)
                            break;
                        SvCUR(sv) = generation; /* (SvCUR not used any more) */
@@ -2179,6 +2187,39 @@ OP *right;
            if (curop != op)
                op->op_private = OPpASSIGN_COMMON;
        }
+       if (right && right->op_type == OP_SPLIT) {
+           OP* tmpop;
+           if ((tmpop = ((LISTOP*)right)->op_first) &&
+               tmpop->op_type == OP_PUSHRE)
+           {
+               PMOP *pm = (PMOP*)tmpop;
+               if (left->op_type == OP_RV2AV &&
+                   !(left->op_private & OPpLVAL_INTRO) &&
+                   !(op->op_private & OPpASSIGN_COMMON) )
+               {
+                   tmpop = ((UNOP*)left)->op_first;
+                   if (tmpop->op_type == OP_GV && !pm->op_pmreplroot) {
+                       pm->op_pmreplroot = (OP*)((GVOP*)tmpop)->op_gv;
+                       pm->op_pmflags |= PMf_ONCE;
+                       tmpop = ((UNOP*)op)->op_first;  /* to list (nulled) */
+                       tmpop = ((UNOP*)tmpop)->op_first; /* to pushmark */
+                       tmpop->op_sibling = Nullop;     /* don't free split */
+                       right->op_next = tmpop->op_next;  /* fix starting loc */
+                       op_free(op);                    /* blow off assign */
+                       return right;
+                   }
+               }
+               else {
+                   if (modcount < 10000 &&
+                     ((LISTOP*)right)->op_last->op_type == OP_CONST)
+                   {
+                       SV *sv = ((SVOP*)((LISTOP*)right)->op_last)->op_sv;
+                       if (SvIVX(sv) == 0)
+                           sv_setiv(sv, modcount+1);
+                   }
+               }
+           }
+       }
        return op;
     }
     if (!right)
@@ -2215,9 +2256,10 @@ OP *op;
        I32 i;
        SV *sv;
        for (i = min_intro_pending; i <= max_intro_pending; i++) {
-           if ((sv = svp[i]) && sv != &sv_undef)
+           if ((sv = svp[i]) && sv != &sv_undef && !SvIVX(sv)) {
                SvIVX(sv) = 999999999;  /* Don't know scope end yet. */
                SvNVX(sv) = (double)cop_seqmax;
+           }
        }
        min_intro_pending = 0;
        comppad_name_fill = max_intro_pending;  /* Needn't search higher */
@@ -2233,7 +2275,7 @@ OP *op;
        cop->op_ppaddr = ppaddr[ OP_NEXTSTATE ];
     }
     cop->op_flags = flags;
-    cop->op_private = 0;
+    cop->op_private = 0 | (flags >> 8);
     cop->op_next = (OP*)cop;
 
     if (label) {
@@ -2295,7 +2337,7 @@ OP* other;
     }
     if (first->op_type == OP_CONST) {
        if (dowarn && (first->op_private & OPpCONST_BARE))
-           warn("Probable precedence problem on %s", op_name[type]);
+           warn("Probable precedence problem on %s", op_desc[type]);
        if ((type == OP_AND) == (SvTRUE(((SVOP*)first)->op_sv))) {
            op_free(first);
            return other;
@@ -2325,7 +2367,7 @@ OP* other;
     logop->op_first = first;
     logop->op_flags = flags | OPf_KIDS;
     logop->op_other = LINKLIST(other);
-    logop->op_private = 1;
+    logop->op_private = 1 | (flags >> 8);
 
     /* establish postfix order */
     logop->op_next = LINKLIST(first);
@@ -2378,7 +2420,7 @@ OP* false;
     condop->op_flags = flags | OPf_KIDS;
     condop->op_true = LINKLIST(true);
     condop->op_false = LINKLIST(false);
-    condop->op_private = 1;
+    condop->op_private = 1 | (flags >> 8);
 
     /* establish postfix order */
     condop->op_next = LINKLIST(first);
@@ -2413,7 +2455,7 @@ OP *right;
     condop->op_flags = OPf_KIDS;
     condop->op_true = LINKLIST(left);
     condop->op_false = LINKLIST(right);
-    condop->op_private = 1;
+    condop->op_private = 1 | (flags >> 8);
 
     left->op_sibling = right;
 
@@ -2542,6 +2584,7 @@ OP *cont;
        loop->op_nextop = op;
 
     op->op_flags |= flags;
+    op->op_private |= (flags >> 8);
     return op;
 }
 
@@ -2561,6 +2604,7 @@ newFOROP(I32 flags,char *label,line_t forline,OP *sv,OP *expr,OP *block,OP *cont
 {
     LOOP *loop;
     int padoff = 0;
+    I32 iterflags = 0;
 
     copline = forline;
     if (sv) {
@@ -2574,12 +2618,16 @@ newFOROP(I32 flags,char *label,line_t forline,OP *sv,OP *expr,OP *block,OP *cont
            sv = Nullop;
        }
        else
-           croak("Can't use %s for loop variable", op_name[sv->op_type]);
+           croak("Can't use %s for loop variable", op_desc[sv->op_type]);
     }
     else {
        sv = newGVOP(OP_GV, 0, defgv);
     }
-    loop = (LOOP*)list(convert(OP_ENTERITER, 0,
+    if (expr->op_type == OP_RV2AV) {
+       expr = scalar(ref(expr, OP_ITER));
+       iterflags |= OPf_STACKED;
+    }
+    loop = (LOOP*)list(convert(OP_ENTERITER, iterflags,
        append_elem(OP_LIST, mod(force_list(expr), OP_GREPSTART),
                    scalar(sv))));
     assert(!loop->op_next);
@@ -2623,23 +2671,25 @@ CV *cv;
        SAVESPTR(curpad);
        curpad = 0;
 
-       if (!SvFLAGS(cv) & SVpcv_CLONED)
+       if (!(SvFLAGS(cv) & SVpcv_CLONED))
            op_free(CvROOT(cv));
        CvROOT(cv) = Nullop;
-       if (CvPADLIST(cv)) {
-           I32 i = AvFILL(CvPADLIST(cv));
-           while (i >= 0) {
-               SV** svp = av_fetch(CvPADLIST(cv), i--, FALSE);
-               if (svp)
-                   SvREFCNT_dec(*svp);
-           }
-           SvREFCNT_dec((SV*)CvPADLIST(cv));
-           CvPADLIST(cv) = Nullav;
-       }
-       SvREFCNT_dec(CvGV(cv));
-       CvGV(cv) = Nullgv;
        LEAVE;
     }
+    SvREFCNT_dec(CvGV(cv));
+    CvGV(cv) = Nullgv;
+    SvREFCNT_dec(CvOUTSIDE(cv));
+    CvOUTSIDE(cv) = Nullcv;
+    if (CvPADLIST(cv)) {
+       I32 i = AvFILL(CvPADLIST(cv));
+       while (i >= 0) {
+           SV** svp = av_fetch(CvPADLIST(cv), i--, FALSE);
+           if (svp)
+               SvREFCNT_dec(*svp);
+       }
+       SvREFCNT_dec((SV*)CvPADLIST(cv));
+       CvPADLIST(cv) = Nullav;
+    }
 }
 
 CV *
@@ -2669,7 +2719,8 @@ CV* proto;
     CvSTASH(cv)                = CvSTASH(proto);
     CvROOT(cv)         = CvROOT(proto);
     CvSTART(cv)                = CvSTART(proto);
-    CvOUTSIDE(cv)      = CvOUTSIDE(proto);
+    if (CvOUTSIDE(proto))
+       CvOUTSIDE(cv)   = (CV*)SvREFCNT_inc((SV*)CvOUTSIDE(proto));
 
     comppad = newAV();
 
@@ -2692,7 +2743,8 @@ CV* proto;
        if (svp[ix] != &sv_undef) {
            char *name = SvPVX(svp[ix]);    /* XXX */
            if (SvFLAGS(svp[ix]) & SVf_FAKE) {  /* lexical from outside? */
-               I32 off = pad_findlex(name,ix,curcop->cop_seq, CvOUTSIDE(proto), cxstack_ix);
+               I32 off = pad_findlex(name,ix,curcop->cop_seq, CvOUTSIDE(proto),
+                                       cxstack_ix);
                if (off != ix)
                    croak("panic: cv_clone: %s", name);
            }
@@ -2717,14 +2769,15 @@ CV* proto;
 }
 
 CV *
-newSUB(floor,op,block)
+newSUB(floor,op,proto,block)
 I32 floor;
 OP *op;
+OP *proto;
 OP *block;
 {
     register CV *cv;
     char *name = op ? SvPVx(cSVOP->op_sv, na) : "__ANON__";
-    GV *gv = gv_fetchpv(name, GV_ADDMULTI, SVt_PVCV);
+    GV* gv = gv_fetchpv(name, GV_ADDMULTI, SVt_PVCV);
     AV* av;
     char *s;
     I32 ix;
@@ -2747,12 +2800,13 @@ OP *block;
        }
     }
     if (cv) {                          /* must reuse cv if autoloaded */
-       if (CvGV(cv)) {
-           assert(SvREFCNT(CvGV(cv)) > 1);
-           SvREFCNT_dec(CvGV(cv));
-       }
+       cv_undef(cv);
        CvOUTSIDE(cv) = CvOUTSIDE(compcv);
+       CvOUTSIDE(compcv) = 0;
        CvPADLIST(cv) = CvPADLIST(compcv);
+       CvPADLIST(compcv) = 0;
+       if (SvREFCNT(compcv) > 1) /* XXX Make closures transit through stub. */
+           CvOUTSIDE(compcv) = (CV*)SvREFCNT_inc((SV*)cv);
        SvREFCNT_dec(compcv);
     }
     else {
@@ -2764,6 +2818,18 @@ OP *block;
     CvGV(cv) = SvREFCNT_inc(gv);
     CvSTASH(cv) = curstash;
 
+    if (proto) {
+       char *p = SvPVx(((SVOP*)proto)->op_sv, na);
+       if (SvPOK(cv) && strNE(SvPV((SV*)cv,na), p))
+           warn("Prototype mismatch: (%s) vs (%s)", SvPV((SV*)cv, na), p);
+       sv_setpv((SV*)cv, p);
+       op_free(proto);
+    }
+
+    if (error_count) {
+       op_free(block);
+       block = Nullop;
+    }
     if (!block) {
        CvROOT(cv) = 0;
        op_free(op);
@@ -2793,8 +2859,9 @@ OP *block;
        s++;
     else
        s = name;
-    if (strEQ(s, "BEGIN")) {
+    if (strEQ(s, "BEGIN") && !error_count) {
        line_t oldline = compiling.cop_line;
+       SV *oldrs = rs;
 
        ENTER;
        SAVESPTR(compiling.cop_filegv);
@@ -2803,21 +2870,16 @@ OP *block;
            beginav = newAV();
        av_push(beginav, (SV *)cv);
        DEBUG_x( dump_sub(gv) );
-       rs = nrs;
-       rslen = nrslen;
-       rschar = nrschar;
-       rspara = (nrslen == 2);
+       rs = SvREFCNT_inc(nrs);
        GvCV(gv) = 0;
        calllist(beginav);
-       rs = "\n";
-       rslen = 1;
-       rschar = '\n';
-       rspara = 0;
+       SvREFCNT_dec(rs);
+       rs = oldrs;
        curcop = &compiling;
        curcop->cop_line = oldline;     /* might have recursed to yylex */
        LEAVE;
     }
-    else if (strEQ(s, "END")) {
+    else if (strEQ(s, "END") && !error_count) {
        if (!endav)
            endav = newAV();
        av_unshift(endav, 1);
@@ -2933,7 +2995,6 @@ OP *block;
     register CV *cv;
     char *name;
     GV *gv;
-    AV* av;
     I32 ix;
 
     if (op)
@@ -2962,11 +3023,6 @@ OP *block;
            SvPADTMP_on(curpad[ix]);
     }
 
-    CvPADLIST(cv) = av = newAV();
-    AvREAL_off(av);
-    av_store(av, 1, SvREFCNT_inc((SV*)comppad));
-    AvFILL(av) = 1;
-
     CvROOT(cv) = newUNOP(OP_LEAVEWRITE, 0, scalarseq(block));
     CvSTART(cv) = LINKLIST(CvROOT(cv));
     CvROOT(cv)->op_next = 0;
@@ -2978,25 +3034,6 @@ OP *block;
 }
 
 OP *
-newMETHOD(ref,name)
-OP *ref;
-OP *name;
-{
-    LOGOP* mop;
-    Newz(1101, mop, 1, LOGOP);
-    mop->op_type = OP_METHOD;
-    mop->op_ppaddr = ppaddr[OP_METHOD];
-    mop->op_first = scalar(ref);
-    mop->op_flags |= OPf_KIDS;
-    mop->op_private = 1;
-    mop->op_other = LINKLIST(name);
-    mop->op_targ = pad_alloc(OP_METHOD, SVs_PADTMP);
-    mop->op_next = LINKLIST(ref);
-    ref->op_next = (OP*)mop;
-    return scalar((OP*)mop);
-}
-
-OP *
 newANONLIST(op)
 OP* op;
 {
@@ -3013,12 +3050,13 @@ OP* op;
 }
 
 OP *
-newANONSUB(floor, block)
+newANONSUB(floor, proto, block)
 I32 floor;
+OP *proto;
 OP *block;
 {
     return newUNOP(OP_REFGEN, 0,
-       newSVOP(OP_ANONCODE, 0, (SV*)newSUB(floor, 0, block)));
+       newSVOP(OP_ANONCODE, 0, (SV*)newSUB(floor, 0, proto, block)));
 }
 
 OP *
@@ -3113,10 +3151,11 @@ OP *o;
 }
 
 OP *
-newCVREF(o)
+newCVREF(flags, o)
+I32 flags;
 OP *o;
 {
-    return newUNOP(OP_RV2CV, 0, scalar(o));
+    return newUNOP(OP_RV2CV, flags, scalar(o));
 }
 
 OP *
@@ -3175,7 +3214,7 @@ OP *op;
     if (op->op_flags & OPf_KIDS) {
        OP *kid = cUNOP->op_first;
        if (kid->op_type != OP_HELEM)
-           croak("%s argument is not a HASH element", op_name[op->op_type]);
+           croak("%s argument is not a HASH element", op_desc[op->op_type]);
        null(kid);
     }
     return op;
@@ -3272,7 +3311,7 @@ register OP *op;
 {
     SVOP *kid = (SVOP*)cUNOP->op_first;
 
-    op->op_private = (hints & HINT_STRICT_REFS);
+    op->op_private |= (hints & HINT_STRICT_REFS);
     if (kid->op_type == OP_CONST) {
        int iscv = (op->op_type==OP_RV2CV)*2;
        GV *gv = 0;
@@ -3394,14 +3433,14 @@ OP *op;
                        gv_fetchpv(name, TRUE, SVt_PVAV) ));
                    if (dowarn)
                        warn("Array @%s missing the @ in argument %d of %s()",
-                           name, numargs, op_name[type]);
+                           name, numargs, op_desc[type]);
                    op_free(kid);
                    kid = newop;
                    kid->op_sibling = sibl;
                    *tokid = kid;
                }
                else if (kid->op_type != OP_RV2AV && kid->op_type != OP_PADAV)
-                   bad_type(numargs, "array", op, kid);
+                   bad_type(numargs, "array", op_desc[op->op_type], kid);
                mod(kid, type);
                break;
            case OA_HVREF:
@@ -3412,14 +3451,14 @@ OP *op;
                        gv_fetchpv(name, TRUE, SVt_PVHV) ));
                    if (dowarn)
                        warn("Hash %%%s missing the %% in argument %d of %s()",
-                           name, numargs, op_name[type]);
+                           name, numargs, op_desc[type]);
                    op_free(kid);
                    kid = newop;
                    kid->op_sibling = sibl;
                    *tokid = kid;
                }
                else if (kid->op_type != OP_RV2HV && kid->op_type != OP_PADHV)
-                   bad_type(numargs, "hash", op, kid);
+                   bad_type(numargs, "hash", op_desc[op->op_type], kid);
                mod(kid, type);
                break;
            case OA_CVREF:
@@ -3460,9 +3499,9 @@ OP *op;
            tokid = &kid->op_sibling;
            kid = kid->op_sibling;
        }
-       op->op_private = numargs;
+       op->op_private |= numargs;
        if (kid)
-           return too_many_arguments(op);
+           return too_many_arguments(op,op_desc[op->op_type]);
        listkids(op);
     }
     else if (opargs[type] & OA_DEFGV) {
@@ -3474,7 +3513,7 @@ OP *op;
        while (oa & OA_OPTIONAL)
            oa >>= 4;
        if (oa && oa != OA_LIST)
-           return too_few_arguments(op);
+           return too_few_arguments(op,op_desc[op->op_type]);
     }
     return op;
 }
@@ -3504,6 +3543,7 @@ OP *op;
     if (op->op_flags & OPf_STACKED) {
        OP* k;
        op = ck_sort(op);
+        kid = cLISTOP->op_first->op_sibling;
        for (k = cLISTOP->op_first->op_sibling->op_next; k; k = k->op_next) {
            kid = k;
        }
@@ -3534,7 +3574,7 @@ OP *op;
 
     kid = cLISTOP->op_first->op_sibling;
     if (!kid || !kid->op_sibling)
-       return too_few_arguments(op);
+       return too_few_arguments(op,op_desc[op->op_type]);
     for (kid = kid->op_sibling; kid; kid = kid->op_sibling)
        mod(kid, OP_GREPSTART);
 
@@ -3611,6 +3651,7 @@ ck_match(op)
 OP *op;
 {
     cPMOP->op_pmflags |= PMf_RUNTIME;
+    cPMOP->op_pmpermflags |= PMf_RUNTIME;
     return op;
 }
 
@@ -3626,7 +3667,7 @@ ck_repeat(op)
 OP *op;
 {
     if (cBINOP->op_first->op_flags & OPf_PARENS) {
-       op->op_private = OPpREPEAT_DOLIST;
+       op->op_private |= OPpREPEAT_DOLIST;
        cBINOP->op_first = force_list(cBINOP->op_first);
     }
     else
@@ -3669,8 +3710,9 @@ OP *
 ck_select(op)
 OP *op;
 {
+    OP* kid;
     if (op->op_flags & OPf_KIDS) {
-       OP *kid = cLISTOP->op_first->op_sibling;        /* get past pushmark */
+       kid = cLISTOP->op_first->op_sibling;    /* get past pushmark */
        if (kid && kid->op_sibling) {
            op->op_type = OP_SSELECT;
            op->op_ppaddr = ppaddr[OP_SSELECT];
@@ -3678,7 +3720,11 @@ OP *op;
            return fold_constants(op);
        }
     }
-    return ck_fun(op);
+    op = ck_fun(op);
+    kid = cLISTOP->op_first->op_sibling;    /* get past pushmark */
+    if (kid && kid->op_type == OP_RV2GV)
+       kid->op_private &= ~HINT_STRICT_REFS;
+    return op;
 }
 
 OP *
@@ -3793,7 +3839,7 @@ OP *op;
     scalar(kid);
 
     if (kid->op_sibling)
-       return too_many_arguments(op);
+       return too_many_arguments(op,op_desc[op->op_type]);
 
     return op;
 }
@@ -3802,16 +3848,117 @@ OP *
 ck_subr(op)
 OP *op;
 {
-    OP *o = ((cUNOP->op_first->op_sibling)
-            ? cUNOP : ((UNOP*)cUNOP->op_first))->op_first->op_sibling;
-
-    if (o->op_type == OP_RV2CV)
-       null(o);                /* disable rv2cv */
-    op->op_private = (hints & HINT_STRICT_REFS);
+    OP *prev = ((cUNOP->op_first->op_sibling)
+            ? cUNOP : ((UNOP*)cUNOP->op_first))->op_first;
+    OP *o = prev->op_sibling;
+    OP *cvop;
+    char *proto = 0;
+    CV *cv = 0;
+    int optional = 0;
+    I32 arg = 0;
+
+    for (cvop = o; cvop->op_sibling; cvop = cvop->op_sibling) ;
+    if (cvop->op_type == OP_RV2CV) {
+       SVOP* tmpop;
+       op->op_private |= (cvop->op_private & OPpENTERSUB_AMPER);
+       null(cvop);             /* disable rv2cv */
+       tmpop = (SVOP*)((UNOP*)cvop)->op_first;
+       if (tmpop->op_type == OP_GV) {
+           cv = GvCV(tmpop->op_sv);
+           if (cv && SvPOK(cv) && !(op->op_private & OPpENTERSUB_AMPER))
+               proto = SvPV((SV*)cv,na);
+       }
+    }
+    op->op_private |= (hints & HINT_STRICT_REFS);
     if (perldb && curstash != debstash)
-       op->op_private |= OPpDEREF_DB;
-    while (o = o->op_sibling)
+       op->op_private |= OPpENTERSUB_DB;
+    while (o != cvop) {
+       if (proto) {
+           switch (*proto) {
+           case '\0':
+               return too_many_arguments(op, CvNAME(cv));
+           case ';':
+               optional = 1;
+               proto++;
+               continue;
+           case '$':
+               proto++;
+               arg++;
+               scalar(o);
+               break;
+           case '%':
+           case '@':
+               list(o);
+               arg++;
+               break;
+           case '&':
+               proto++;
+               arg++;
+               if (o->op_type != OP_REFGEN && o->op_type != OP_UNDEF)
+                   bad_type(arg, "block", CvNAME(cv), o);
+               break;
+           case '*':
+               proto++;
+               arg++;
+               if (o->op_type == OP_RV2GV)
+                   goto wrapref;
+               {
+                   OP* kid = o;
+                   o = newUNOP(OP_RV2GV, 0, kid);
+                   o->op_sibling = kid->op_sibling;
+                   kid->op_sibling = 0;
+                   prev->op_sibling = o;
+               }
+               goto wrapref;
+           case '\\':
+               proto++;
+               arg++;
+               switch (*proto++) {
+               case '*':
+                   if (o->op_type != OP_RV2GV)
+                       bad_type(arg, "symbol", CvNAME(cv), o);
+                   goto wrapref;
+               case '&':
+                   if (o->op_type != OP_RV2CV)
+                       bad_type(arg, "sub", CvNAME(cv), o);
+                   goto wrapref;
+               case '$':
+                   if (o->op_type != OP_RV2SV && o->op_type != OP_PADSV)
+                       bad_type(arg, "scalar", CvNAME(cv), o);
+                   goto wrapref;
+               case '@':
+                   if (o->op_type != OP_RV2AV && o->op_type != OP_PADAV)
+                       bad_type(arg, "array", CvNAME(cv), o);
+                   goto wrapref;
+               case '%':
+                   if (o->op_type != OP_RV2HV && o->op_type != OP_PADHV)
+                       bad_type(arg, "hash", CvNAME(cv), o);
+                 wrapref:
+                   {
+                       OP* kid = o;
+                       o = newUNOP(OP_REFGEN, 0, kid);
+                       o->op_sibling = kid->op_sibling;
+                       kid->op_sibling = 0;
+                       prev->op_sibling = o;
+                   }
+                   break;
+               default: goto oops;
+               }
+               break;
+           default:
+             oops:
+               croak("Malformed prototype for %s: %s",
+                       CvNAME(cv),SvPV((SV*)cv,na));
+           }
+       }
+       else
+           list(o);
        mod(o, OP_ENTERSUB);
+       prev = o;
+       o = o->op_sibling;
+    }
+    if (proto && !optional && *proto == '$')
+       return too_few_arguments(op, CvNAME(cv));
     return op;
 }
 
@@ -3854,12 +4001,14 @@ register OP* o;
     for (; o; o = o->op_next) {
        if (o->op_seq)
            break;
+       if (!op_seqmax)
+           op_seqmax++;
        op = o;
        switch (o->op_type) {
        case OP_NEXTSTATE:
        case OP_DBSTATE:
            curcop = ((COP*)o);         /* for warnings */
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            break;
 
        case OP_CONCAT:
@@ -3872,11 +4021,11 @@ register OP* o;
        case OP_QUOTEMETA:
            if (o->op_next->op_type == OP_STRINGIFY)
                null(o->op_next);
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            break;
        case OP_STUB:
            if ((o->op_flags & (OPf_KNOW|OPf_LIST)) != (OPf_KNOW|OPf_LIST)) {
-               o->op_seq = ++op_seqmax;
+               o->op_seq = op_seqmax++;
                break;  /* Scalar stub must produce undef.  List stub is noop */
            }
            goto nothin;
@@ -3892,7 +4041,7 @@ register OP* o;
                oldop->op_next = o->op_next;
                continue;
            }
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            break;
 
        case OP_GV:
@@ -3929,25 +4078,25 @@ register OP* o;
                    GvAVn((GV*)(((SVOP*)o)->op_sv));
                }
            }
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            break;
 
        case OP_MAPWHILE:
        case OP_GREPWHILE:
        case OP_AND:
        case OP_OR:
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            peep(cLOGOP->op_other);
            break;
 
        case OP_COND_EXPR:
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            peep(cCONDOP->op_true);
            peep(cCONDOP->op_false);
            break;
 
        case OP_ENTERLOOP:
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            peep(cLOOP->op_redoop);
            peep(cLOOP->op_nextop);
            peep(cLOOP->op_lastop);
@@ -3955,12 +4104,12 @@ register OP* o;
 
        case OP_MATCH:
        case OP_SUBST:
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            peep(cPMOP->op_pmreplstart);
            break;
 
        case OP_EXEC:
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            if (dowarn && o->op_next && o->op_next->op_type == OP_NEXTSTATE) {
                if (o->op_next->op_sibling &&
                        o->op_next->op_sibling->op_type != OP_DIE) {
@@ -3974,7 +4123,7 @@ register OP* o;
            }
            break;
        default:
-           o->op_seq = ++op_seqmax;
+           o->op_seq = op_seqmax++;
            break;
        }
        oldop = o;