This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
format "..." bug
[perl5.git] / dump.c
diff --git a/dump.c b/dump.c
index e74c8c4..85cdddb 100644 (file)
--- a/dump.c
+++ b/dump.c
 #include "EXTERN.h"
 #include "perl.h"
 
-#ifndef DEBUGGING
+#ifndef DBL_DIG
+#define DBL_DIG        15   /* A guess that works lots of places */
+#endif
+
 void
-dump_all()
+dump_indent(I32 level, PerlIO *file, const char* pat, ...)
 {
+    dTHR;
+    va_list args;
+    
+    va_start(args, pat);
+    PerlIO_printf(file, "%*s", level*PL_dumpindent, "");
+    PerlIO_vprintf(file, pat, args);
+    va_end(args);
 }
-#else  /* Rest of file is for DEBUGGING */
-
-#ifdef I_STDARG
-static void dump(char *pat, ...);
-#else
-static void dump();
-#endif
 
 void
-dump_all()
+dump_all(void)
 {
+    dTHR;
     PerlIO_setlinebuf(Perl_debug_log);
-    if (main_root)
-       dump_op(main_root);
-    dump_packsubs(defstash);
+    if (PL_main_root)
+       op_dump(PL_main_root);
+    dump_packsubs(PL_defstash);
 }
 
 void
-dump_packsubs(stash)
-HV* stash;
+dump_packsubs(HV *stash)
 {
+    dTHR;
     I32        i;
     HE *entry;
 
@@ -50,276 +54,532 @@ HV* stash;
        for (entry = HvARRAY(stash)[i]; entry; entry = HeNEXT(entry)) {
            GV *gv = (GV*)HeVAL(entry);
            HV *hv;
+           if (SvTYPE(gv) != SVt_PVGV || !GvGP(gv))
+               continue;
            if (GvCVu(gv))
                dump_sub(gv);
            if (GvFORM(gv))
                dump_form(gv);
            if (HeKEY(entry)[HeKLEN(entry)-1] == ':' &&
-             (hv = GvHV(gv)) && HvNAME(hv) && hv != defstash)
+             (hv = GvHV(gv)) && HvNAME(hv) && hv != PL_defstash)
                dump_packsubs(hv);              /* nested package */
        }
     }
 }
 
 void
-dump_sub(gv)
-GV* gv;
+dump_sub(GV *gv)
 {
     SV *sv = sv_newmortal();
 
     gv_fullname3(sv, gv, Nullch);
-    dump("\nSUB %s = ", SvPVX(sv));
+    dump_indent(0, Perl_debug_log, "\nSUB %s = ", SvPVX(sv));
     if (CvXSUB(GvCV(gv)))
-       dump("(xsub 0x%x %d)\n",
+       dump_indent(0, Perl_debug_log, "(xsub 0x%x %d)\n",
            (long)CvXSUB(GvCV(gv)),
            CvXSUBANY(GvCV(gv)).any_i32);
     else if (CvROOT(GvCV(gv)))
-       dump_op(CvROOT(GvCV(gv)));
+       op_dump(CvROOT(GvCV(gv)));
     else
-       dump("<undef>\n");
+       dump_indent(0, Perl_debug_log, "<undef>\n");
 }
 
 void
-dump_form(gv)
-GV* gv;
+dump_form(GV *gv)
 {
     SV *sv = sv_newmortal();
 
     gv_fullname3(sv, gv, Nullch);
-    dump("\nFORMAT %s = ", SvPVX(sv));
+    dump_indent(0, Perl_debug_log, "\nFORMAT %s = ", SvPVX(sv));
     if (CvROOT(GvFORM(gv)))
-       dump_op(CvROOT(GvFORM(gv)));
+       op_dump(CvROOT(GvFORM(gv)));
+    else
+       dump_indent(0, Perl_debug_log, "<undef>\n");
+}
+
+void
+dump_eval(void)
+{
+    op_dump(PL_eval_root);
+}
+
+char *
+pv_display(SV *sv, char *pv, STRLEN cur, STRLEN len, STRLEN pvlim)
+{
+    int truncated = 0;
+    int nul_terminated = len > cur && pv[cur] == '\0';
+
+    sv_setpvn(sv, "\"", 1);
+    for (; cur--; pv++) {
+       if (pvlim && SvCUR(sv) >= pvlim) {
+            truncated++;
+           break;
+        }
+        if (isPRINT(*pv)) {
+            switch (*pv) {
+           case '\t': sv_catpvn(sv, "\\t", 2);  break;
+           case '\n': sv_catpvn(sv, "\\n", 2);  break;
+           case '\r': sv_catpvn(sv, "\\r", 2);  break;
+           case '\f': sv_catpvn(sv, "\\f", 2);  break;
+           case '"':  sv_catpvn(sv, "\\\"", 2); break;
+           case '\\': sv_catpvn(sv, "\\\\", 2); break;
+           default:   sv_catpvn(sv, pv, 1);     break;
+            }
+        }
+       else {
+           if (cur && isDIGIT(*(pv+1)))
+               sv_catpvf(sv, "\\%03o", *pv);
+           else
+               sv_catpvf(sv, "\\%o", *pv);
+        }
+    }
+    sv_catpvn(sv, "\"", 1);
+    if (truncated)
+       sv_catpvn(sv, "...", 3);
+    if (nul_terminated)
+       sv_catpvn(sv, "\\0", 2);
+
+    return SvPVX(sv);
+}
+
+char *
+sv_peek(SV *sv)
+{
+    SV *t = sv_newmortal();
+    STRLEN prevlen;
+    int unref = 0;
+
+    sv_setpvn(t, "", 0);
+  retry:
+    if (!sv) {
+       sv_catpv(t, "VOID");
+       goto finish;
+    }
+    else if (sv == (SV*)0x55555555 || SvTYPE(sv) == 'U') {
+       sv_catpv(t, "WILD");
+       goto finish;
+    }
+    else if (sv == &PL_sv_undef || sv == &PL_sv_no || sv == &PL_sv_yes) {
+       if (sv == &PL_sv_undef) {
+           sv_catpv(t, "SV_UNDEF");
+           if (!(SvFLAGS(sv) & (SVf_OK|SVf_OOK|SVs_OBJECT|
+                                SVs_GMG|SVs_SMG|SVs_RMG)) &&
+               SvREADONLY(sv))
+               goto finish;
+       }
+       else if (sv == &PL_sv_no) {
+           sv_catpv(t, "SV_NO");
+           if (!(SvFLAGS(sv) & (SVf_ROK|SVf_OOK|SVs_OBJECT|
+                                SVs_GMG|SVs_SMG|SVs_RMG)) &&
+               !(~SvFLAGS(sv) & (SVf_POK|SVf_NOK|SVf_READONLY|
+                                 SVp_POK|SVp_NOK)) &&
+               SvCUR(sv) == 0 &&
+               SvNVX(sv) == 0.0)
+               goto finish;
+       }
+       else {
+           sv_catpv(t, "SV_YES");
+           if (!(SvFLAGS(sv) & (SVf_ROK|SVf_OOK|SVs_OBJECT|
+                                SVs_GMG|SVs_SMG|SVs_RMG)) &&
+               !(~SvFLAGS(sv) & (SVf_POK|SVf_NOK|SVf_READONLY|
+                                 SVp_POK|SVp_NOK)) &&
+               SvCUR(sv) == 1 &&
+               SvPVX(sv) && *SvPVX(sv) == '1' &&
+               SvNVX(sv) == 1.0)
+               goto finish;
+       }
+       sv_catpv(t, ":");
+    }
+    else if (SvREFCNT(sv) == 0) {
+       sv_catpv(t, "(");
+       unref++;
+    }
+    if (SvROK(sv)) {
+       sv_catpv(t, "\\");
+       if (SvCUR(t) + unref > 10) {
+           SvCUR(t) = unref + 3;
+           *SvEND(t) = '\0';
+           sv_catpv(t, "...");
+           goto finish;
+       }
+       sv = (SV*)SvRV(sv);
+       goto retry;
+    }
+    switch (SvTYPE(sv)) {
+    default:
+       sv_catpv(t, "FREED");
+       goto finish;
+
+    case SVt_NULL:
+       sv_catpv(t, "UNDEF");
+       goto finish;
+    case SVt_IV:
+       sv_catpv(t, "IV");
+       break;
+    case SVt_NV:
+       sv_catpv(t, "NV");
+       break;
+    case SVt_RV:
+       sv_catpv(t, "RV");
+       break;
+    case SVt_PV:
+       sv_catpv(t, "PV");
+       break;
+    case SVt_PVIV:
+       sv_catpv(t, "PVIV");
+       break;
+    case SVt_PVNV:
+       sv_catpv(t, "PVNV");
+       break;
+    case SVt_PVMG:
+       sv_catpv(t, "PVMG");
+       break;
+    case SVt_PVLV:
+       sv_catpv(t, "PVLV");
+       break;
+    case SVt_PVAV:
+       sv_catpv(t, "AV");
+       break;
+    case SVt_PVHV:
+       sv_catpv(t, "HV");
+       break;
+    case SVt_PVCV:
+       if (CvGV(sv))
+           sv_catpvf(t, "CV(%s)", GvNAME(CvGV(sv)));
+       else
+           sv_catpv(t, "CV()");
+       goto finish;
+    case SVt_PVGV:
+       sv_catpv(t, "GV");
+       break;
+    case SVt_PVBM:
+       sv_catpv(t, "BM");
+       break;
+    case SVt_PVFM:
+       sv_catpv(t, "FM");
+       break;
+    case SVt_PVIO:
+       sv_catpv(t, "IO");
+       break;
+    }
+
+    if (SvPOKp(sv)) {
+       if (!SvPVX(sv))
+           sv_catpv(t, "(null)");
+       else {
+           SV *tmp = newSVpv("", 0);
+           sv_catpv(t, "(");
+           if (SvOOK(sv))
+               sv_catpvf(t, "[%s]", pv_display(tmp, SvPVX(sv)-SvIVX(sv), SvIVX(sv), 0, 127));
+           sv_catpvf(t, "%s)", pv_display(tmp, SvPVX(sv), SvCUR(sv), SvLEN(sv), 127));
+           SvREFCNT_dec(tmp);
+       }
+    }
+    else if (SvNOKp(sv)) {
+       SET_NUMERIC_STANDARD();
+       sv_catpvf(t, "(%g)",SvNVX(sv));
+    }
+    else if (SvIOKp(sv))
+       sv_catpvf(t, "(%ld)",(long)SvIVX(sv));
     else
-       dump("<undef>\n");
+       sv_catpv(t, "()");
+    
+  finish:
+    if (unref) {
+       while (unref--)
+           sv_catpv(t, ")");
+    }
+    return SvPV(t, PL_na);
 }
 
 void
-dump_eval()
+do_pmop_dump(I32 level, PerlIO *file, PMOP *pm)
 {
-    dump_op(eval_root);
+    char ch;
+
+    if (!pm) {
+       dump_indent(level, file, "{}\n");
+       return;
+    }
+    dump_indent(level, file, "{\n");
+    level++;
+    if (pm->op_pmflags & PMf_ONCE)
+       ch = '?';
+    else
+       ch = '/';
+    if (pm->op_pmregexp)
+       dump_indent(level, file, "PMf_PRE %c%s%c%s\n",
+            ch, pm->op_pmregexp->precomp, ch,
+            (pm->op_private & OPpRUNTIME) ? " (RUNTIME)" : "");
+    else
+       dump_indent(level, file, "PMf_PRE (RUNTIME)\n");
+    if (pm->op_type != OP_PUSHRE && pm->op_pmreplroot) {
+       dump_indent(level, file, "PMf_REPL = ");
+       op_dump(pm->op_pmreplroot);
+    }
+    if (pm->op_pmflags || (pm->op_pmregexp && pm->op_pmregexp->check_substr)) {
+       SV *tmpsv = newSVpv("", 0);
+       if (pm->op_pmdynflags & PMdf_USED)
+           sv_catpv(tmpsv, ",USED");
+       if (pm->op_pmdynflags & PMdf_TAINTED)
+           sv_catpv(tmpsv, ",TAINTED");
+       if (pm->op_pmflags & PMf_ONCE)
+           sv_catpv(tmpsv, ",ONCE");
+       if (pm->op_pmregexp && pm->op_pmregexp->check_substr
+           && !(pm->op_pmregexp->reganch & ROPT_NOSCAN))
+           sv_catpv(tmpsv, ",SCANFIRST");
+       if (pm->op_pmregexp && pm->op_pmregexp->check_substr
+           && pm->op_pmregexp->reganch & ROPT_CHECK_ALL)
+           sv_catpv(tmpsv, ",ALL");
+       if (pm->op_pmflags & PMf_SKIPWHITE)
+           sv_catpv(tmpsv, ",SKIPWHITE");
+       if (pm->op_pmflags & PMf_CONST)
+           sv_catpv(tmpsv, ",CONST");
+       if (pm->op_pmflags & PMf_KEEP)
+           sv_catpv(tmpsv, ",KEEP");
+       if (pm->op_pmflags & PMf_GLOBAL)
+           sv_catpv(tmpsv, ",GLOBAL");
+       if (pm->op_pmflags & PMf_CONTINUE)
+           sv_catpv(tmpsv, ",CONTINUE");
+       if (pm->op_pmflags & PMf_RETAINT)
+           sv_catpv(tmpsv, ",RETAINT");
+       if (pm->op_pmflags & PMf_EVAL)
+           sv_catpv(tmpsv, ",EVAL");
+       dump_indent(level, file, "PMFLAGS = (%s)\n", SvCUR(tmpsv) ? SvPVX(tmpsv) + 1 : "");
+       SvREFCNT_dec(tmpsv);
+    }
+
+    dump_indent(level-1, file, "}\n");
 }
 
 void
-dump_op(op)
-register OP *op;
+pmop_dump(PMOP *pm)
 {
-    SV *tmpsv;
+    do_pmop_dump(0, Perl_debug_log, pm);
+}
 
-    dump("{\n");
-    if (op->op_seq)
-       PerlIO_printf(Perl_debug_log, "%-4d", op->op_seq);
+void
+do_op_dump(I32 level, PerlIO *file, OP *o)
+{
+    dTHR;
+    dump_indent(level, file, "{\n");
+    level++;
+    if (o->op_seq)
+       PerlIO_printf(file, "%-4d", o->op_seq);
     else
-       PerlIO_printf(Perl_debug_log, "    ");
-    dump("TYPE = %s  ===> ", op_name[op->op_type]);
-    if (op->op_next) {
-       if (op->op_seq)
-           PerlIO_printf(Perl_debug_log, "%d\n", op->op_next->op_seq);
+       PerlIO_printf(file, "    ");
+    PerlIO_printf(file, "%*sTYPE = %s  ===> ", PL_dumpindent*level-4, "", PL_op_name[o->op_type]);
+    if (o->op_next) {
+       if (o->op_seq)
+           PerlIO_printf(file, "%d\n", o->op_next->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "(%d)\n", op->op_next->op_seq);
+           PerlIO_printf(file, "(%d)\n", o->op_next->op_seq);
     }
     else
-       PerlIO_printf(Perl_debug_log, "DONE\n");
-    dumplvl++;
-    if (op->op_targ) {
-       if (op->op_type == OP_NULL)
-           dump("  (was %s)\n", op_name[op->op_targ]);
+       PerlIO_printf(file, "DONE\n");
+    if (o->op_targ) {
+       if (o->op_type == OP_NULL)
+           dump_indent(level, file, "  (was %s)\n", PL_op_name[o->op_targ]);
        else
-           dump("TARG = %d\n", op->op_targ);
+           dump_indent(level, file, "TARG = %d\n", o->op_targ);
     }
 #ifdef DUMPADDR
-    dump("ADDR = 0x%lx => 0x%lx\n",op, op->op_next);
+    dump_indent(level, file, "ADDR = 0x%lx => 0x%lx\n",o, o->op_next);
 #endif
-    if (op->op_flags) {
-       *buf = '\0';
-       switch (op->op_flags & OPf_WANT) {
+    if (o->op_flags) {
+       SV *tmpsv = newSVpv("", 0);
+       switch (o->op_flags & OPf_WANT) {
        case OPf_WANT_VOID:
-           (void)strcat(buf,"VOID,");
+           sv_catpv(tmpsv, ",VOID");
            break;
        case OPf_WANT_SCALAR:
-           (void)strcat(buf,"SCALAR,");
+           sv_catpv(tmpsv, ",SCALAR");
            break;
        case OPf_WANT_LIST:
-           (void)strcat(buf,"LIST,");
+           sv_catpv(tmpsv, ",LIST");
            break;
        default:
-           (void)strcat(buf,"UNKNOWN,");
+           sv_catpv(tmpsv, ",UNKNOWN");
            break;
        }
-       if (op->op_flags & OPf_KIDS)
-           (void)strcat(buf,"KIDS,");
-       if (op->op_flags & OPf_PARENS)
-           (void)strcat(buf,"PARENS,");
-       if (op->op_flags & OPf_STACKED)
-           (void)strcat(buf,"STACKED,");
-       if (op->op_flags & OPf_REF)
-           (void)strcat(buf,"REF,");
-       if (op->op_flags & OPf_MOD)
-           (void)strcat(buf,"MOD,");
-       if (op->op_flags & OPf_SPECIAL)
-           (void)strcat(buf,"SPECIAL,");
-       if (*buf)
-           buf[strlen(buf)-1] = '\0';
-       dump("FLAGS = (%s)\n",buf);
-    }
-    if (op->op_private) {
-       *buf = '\0';
-       if (op->op_type == OP_AASSIGN) {
-           if (op->op_private & OPpASSIGN_COMMON)
-               (void)strcat(buf,"COMMON,");
+       if (o->op_flags & OPf_KIDS)
+           sv_catpv(tmpsv, ",KIDS");
+       if (o->op_flags & OPf_PARENS)
+           sv_catpv(tmpsv, ",PARENS");
+       if (o->op_flags & OPf_STACKED)
+           sv_catpv(tmpsv, ",STACKED");
+       if (o->op_flags & OPf_REF)
+           sv_catpv(tmpsv, ",REF");
+       if (o->op_flags & OPf_MOD)
+           sv_catpv(tmpsv, ",MOD");
+       if (o->op_flags & OPf_SPECIAL)
+           sv_catpv(tmpsv, ",SPECIAL");
+       dump_indent(level, file, "FLAGS = (%s)\n", SvCUR(tmpsv) ? SvPVX(tmpsv) + 1 : "");
+       SvREFCNT_dec(tmpsv);
+    }
+    if (o->op_private) {
+       SV *tmpsv = newSVpv("", 0);
+       if (o->op_type == OP_AASSIGN) {
+           if (o->op_private & OPpASSIGN_COMMON)
+               sv_catpv(tmpsv, ",COMMON");
        }
-       else if (op->op_type == OP_SASSIGN) {
-           if (op->op_private & OPpASSIGN_BACKWARDS)
-               (void)strcat(buf,"BACKWARDS,");
+       else if (o->op_type == OP_SASSIGN) {
+           if (o->op_private & OPpASSIGN_BACKWARDS)
+               sv_catpv(tmpsv, ",BACKWARDS");
        }
-       else if (op->op_type == OP_TRANS) {
-           if (op->op_private & OPpTRANS_SQUASH)
-               (void)strcat(buf,"SQUASH,");
-           if (op->op_private & OPpTRANS_DELETE)
-               (void)strcat(buf,"DELETE,");
-           if (op->op_private & OPpTRANS_COMPLEMENT)
-               (void)strcat(buf,"COMPLEMENT,");
+       else if (o->op_type == OP_TRANS) {
+           if (o->op_private & OPpTRANS_SQUASH)
+               sv_catpv(tmpsv, ",SQUASH");
+           if (o->op_private & OPpTRANS_DELETE)
+               sv_catpv(tmpsv, ",DELETE");
+           if (o->op_private & OPpTRANS_COMPLEMENT)
+               sv_catpv(tmpsv, ",COMPLEMENT");
        }
-       else if (op->op_type == OP_REPEAT) {
-           if (op->op_private & OPpREPEAT_DOLIST)
-               (void)strcat(buf,"DOLIST,");
+       else if (o->op_type == OP_REPEAT) {
+           if (o->op_private & OPpREPEAT_DOLIST)
+               sv_catpv(tmpsv, ",DOLIST");
        }
-       else if (op->op_type == OP_ENTERSUB ||
-                op->op_type == OP_RV2SV ||
-                op->op_type == OP_RV2AV ||
-                op->op_type == OP_RV2HV ||
-                op->op_type == OP_RV2GV ||
-                op->op_type == OP_AELEM ||
-                op->op_type == OP_HELEM )
+       else if (o->op_type == OP_ENTERSUB ||
+                o->op_type == OP_RV2SV ||
+                o->op_type == OP_RV2AV ||
+                o->op_type == OP_RV2HV ||
+                o->op_type == OP_RV2GV ||
+                o->op_type == OP_AELEM ||
+                o->op_type == OP_HELEM )
        {
-           if (op->op_type == OP_ENTERSUB) {
-               if (op->op_private & OPpENTERSUB_AMPER)
-                   (void)strcat(buf,"AMPER,");
-               if (op->op_private & OPpENTERSUB_DB)
-                   (void)strcat(buf,"DB,");
+           if (o->op_type == OP_ENTERSUB) {
+               if (o->op_private & OPpENTERSUB_AMPER)
+                   sv_catpv(tmpsv, ",AMPER");
+               if (o->op_private & OPpENTERSUB_DB)
+                   sv_catpv(tmpsv, ",DB");
            }
-           switch (op->op_private & OPpDEREF) {
+           switch (o->op_private & OPpDEREF) {
            case OPpDEREF_SV:
-               (void)strcat(buf, "SV,");
+               sv_catpv(tmpsv, ",SV");
                break;
            case OPpDEREF_AV:
-               (void)strcat(buf, "AV,");
+               sv_catpv(tmpsv, ",AV");
                break;
            case OPpDEREF_HV:
-               (void)strcat(buf, "HV,");
+               sv_catpv(tmpsv, ",HV");
                break;
            }
-           if (op->op_type == OP_AELEM || op->op_type == OP_HELEM) {
-               if (op->op_private & OPpLVAL_DEFER)
-                   (void)strcat(buf,"LVAL_DEFER,");
+           if (o->op_type == OP_AELEM || o->op_type == OP_HELEM) {
+               if (o->op_private & OPpLVAL_DEFER)
+                   sv_catpv(tmpsv, ",LVAL_DEFER");
            }
            else {
-               if (op->op_private & HINT_STRICT_REFS)
-                   (void)strcat(buf,"STRICT_REFS,");
+               if (o->op_private & HINT_STRICT_REFS)
+                   sv_catpv(tmpsv, ",STRICT_REFS");
            }
        }
-       else if (op->op_type == OP_CONST) {
-           if (op->op_private & OPpCONST_BARE)
-               (void)strcat(buf,"BARE,");
-       }
-       else if (op->op_type == OP_FLIP) {
-           if (op->op_private & OPpFLIP_LINENUM)
-               (void)strcat(buf,"LINENUM,");
+       else if (o->op_type == OP_CONST) {
+           if (o->op_private & OPpCONST_BARE)
+               sv_catpv(tmpsv, ",BARE");
        }
-       else if (op->op_type == OP_FLOP) {
-           if (op->op_private & OPpFLIP_LINENUM)
-               (void)strcat(buf,"LINENUM,");
+       else if (o->op_type == OP_FLIP) {
+           if (o->op_private & OPpFLIP_LINENUM)
+               sv_catpv(tmpsv, ",LINENUM");
        }
-       if (op->op_flags & OPf_MOD && op->op_private & OPpLVAL_INTRO)
-           (void)strcat(buf,"INTRO,");
-       if (*buf) {
-           buf[strlen(buf)-1] = '\0';
-           dump("PRIVATE = (%s)\n",buf);
+       else if (o->op_type == OP_FLOP) {
+           if (o->op_private & OPpFLIP_LINENUM)
+               sv_catpv(tmpsv, ",LINENUM");
        }
+       if (o->op_flags & OPf_MOD && o->op_private & OPpLVAL_INTRO)
+           sv_catpv(tmpsv, ",INTRO");
+       if (SvCUR(tmpsv))
+           dump_indent(level, file, "PRIVATE = (%s)\n", SvPVX(tmpsv) + 1);
+       SvREFCNT_dec(tmpsv);
     }
 
-    switch (op->op_type) {
+    switch (o->op_type) {
     case OP_GVSV:
     case OP_GV:
-       if (cGVOP->op_gv) {
+       if (cGVOPo->op_gv) {
+           SV *tmpsv = NEWSV(0,0);
            ENTER;
-           tmpsv = NEWSV(0,0);
            SAVEFREESV(tmpsv);
-           gv_fullname3(tmpsv, cGVOP->op_gv, Nullch);
-           dump("GV = %s\n", SvPV(tmpsv, na));
+           gv_fullname3(tmpsv, cGVOPo->op_gv, Nullch);
+           dump_indent(level, file, "GV = %s\n", SvPV(tmpsv, PL_na));
            LEAVE;
        }
        else
-           dump("GV = NULL\n");
+           dump_indent(level, file, "GV = NULL\n");
        break;
     case OP_CONST:
-       dump("SV = %s\n", SvPEEK(cSVOP->op_sv));
+       dump_indent(level, file, "SV = %s\n", SvPEEK(cSVOPo->op_sv));
        break;
     case OP_NEXTSTATE:
     case OP_DBSTATE:
-       if (cCOP->cop_line)
-           dump("LINE = %d\n",cCOP->cop_line);
-       if (cCOP->cop_label)
-           dump("LABEL = \"%s\"\n",cCOP->cop_label);
+       if (cCOPo->cop_line)
+           dump_indent(level, file, "LINE = %d\n",cCOPo->cop_line);
+       if (cCOPo->cop_label)
+           dump_indent(level, file, "LABEL = \"%s\"\n",cCOPo->cop_label);
        break;
     case OP_ENTERLOOP:
-       dump("REDO ===> ");
-       if (cLOOP->op_redoop)
-           PerlIO_printf(Perl_debug_log, "%d\n", cLOOP->op_redoop->op_seq);
+       dump_indent(level, file, "REDO ===> ");
+       if (cLOOPo->op_redoop)
+           PerlIO_printf(file, "%d\n", cLOOPo->op_redoop->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "DONE\n");
-       dump("NEXT ===> ");
-       if (cLOOP->op_nextop)
-           PerlIO_printf(Perl_debug_log, "%d\n", cLOOP->op_nextop->op_seq);
+           PerlIO_printf(file, "DONE\n");
+       dump_indent(level, file, "NEXT ===> ");
+       if (cLOOPo->op_nextop)
+           PerlIO_printf(file, "%d\n", cLOOPo->op_nextop->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "DONE\n");
-       dump("LAST ===> ");
-       if (cLOOP->op_lastop)
-           PerlIO_printf(Perl_debug_log, "%d\n", cLOOP->op_lastop->op_seq);
+           PerlIO_printf(file, "DONE\n");
+       dump_indent(level, file, "LAST ===> ");
+       if (cLOOPo->op_lastop)
+           PerlIO_printf(file, "%d\n", cLOOPo->op_lastop->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "DONE\n");
+           PerlIO_printf(file, "DONE\n");
        break;
     case OP_COND_EXPR:
-       dump("TRUE ===> ");
-       if (cCONDOP->op_true)
-           PerlIO_printf(Perl_debug_log, "%d\n", cCONDOP->op_true->op_seq);
+       dump_indent(level, file, "TRUE ===> ");
+       if (cCONDOPo->op_true)
+           PerlIO_printf(file, "%d\n", cCONDOPo->op_true->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "DONE\n");
-       dump("FALSE ===> ");
-       if (cCONDOP->op_false)
-           PerlIO_printf(Perl_debug_log, "%d\n", cCONDOP->op_false->op_seq);
+           PerlIO_printf(file, "DONE\n");
+       dump_indent(level, file, "FALSE ===> ");
+       if (cCONDOPo->op_false)
+           PerlIO_printf(file, "%d\n", cCONDOPo->op_false->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "DONE\n");
+           PerlIO_printf(file, "DONE\n");
        break;
     case OP_MAPWHILE:
     case OP_GREPWHILE:
     case OP_OR:
     case OP_AND:
-       dump("OTHER ===> ");
-       if (cLOGOP->op_other)
-           PerlIO_printf(Perl_debug_log, "%d\n", cLOGOP->op_other->op_seq);
+       dump_indent(level, file, "OTHER ===> ");
+       if (cLOGOPo->op_other)
+           PerlIO_printf(file, "%d\n", cLOGOPo->op_other->op_seq);
        else
-           PerlIO_printf(Perl_debug_log, "DONE\n");
+           PerlIO_printf(file, "DONE\n");
        break;
     case OP_PUSHRE:
     case OP_MATCH:
+    case OP_QR:
     case OP_SUBST:
-       dump_pm((PMOP*)op);
+       do_pmop_dump(level, file, cPMOPo);
        break;
     default:
        break;
     }
-    if (op->op_flags & OPf_KIDS) {
+    if (o->op_flags & OPf_KIDS) {
        OP *kid;
-       for (kid = cUNOP->op_first; kid; kid = kid->op_sibling)
-           dump_op(kid);
+       for (kid = cUNOPo->op_first; kid; kid = kid->op_sibling)
+           do_op_dump(level, file, kid);
     }
-    dumplvl--;
-    dump("}\n");
+    dump_indent(level-1, file, "}\n");
 }
 
 void
-dump_gv(gv)
-register GV *gv;
+op_dump(OP *o)
+{
+    do_op_dump(0, Perl_debug_log, o);
+}
+
+void
+gv_dump(GV *gv)
 {
     SV *sv;
 
@@ -328,115 +588,535 @@ register GV *gv;
        return;
     }
     sv = sv_newmortal();
-    dumplvl++;
     PerlIO_printf(Perl_debug_log, "{\n");
     gv_fullname3(sv, gv, Nullch);
-    dump("GV_NAME = %s", SvPVX(sv));
+    dump_indent(1, Perl_debug_log, "GV_NAME = %s", SvPVX(sv));
     if (gv != GvEGV(gv)) {
        gv_efullname3(sv, GvEGV(gv), Nullch);
-       dump("-> %s", SvPVX(sv));
+       dump_indent(1, Perl_debug_log, "-> %s", SvPVX(sv));
     }
-    dump("\n");
-    dumplvl--;
-    dump("}\n");
+    PerlIO_putc(Perl_debug_log, '\n');
+    dump_indent(0, Perl_debug_log, "}\n");
 }
 
 void
-dump_pm(pm)
-register PMOP *pm;
+do_magic_dump(I32 level, PerlIO *file, MAGIC *mg, I32 nest, I32 maxnest, bool dumpops, STRLEN pvlim)
 {
-    char ch;
+    for (; mg; mg = mg->mg_moremagic) {
+       dump_indent(level, file, "  MAGIC = 0x%lx\n", (long)mg);
+       if (mg->mg_virtual) {
+            MGVTBL *v = mg->mg_virtual;
+           char *s = 0;
+           if      (v == &PL_vtbl_sv)         s = "sv";
+            else if (v == &PL_vtbl_env)        s = "env";
+            else if (v == &PL_vtbl_envelem)    s = "envelem";
+            else if (v == &PL_vtbl_sig)        s = "sig";
+            else if (v == &PL_vtbl_sigelem)    s = "sigelem";
+            else if (v == &PL_vtbl_pack)       s = "pack";
+            else if (v == &PL_vtbl_packelem)   s = "packelem";
+            else if (v == &PL_vtbl_dbline)     s = "dbline";
+            else if (v == &PL_vtbl_isa)        s = "isa";
+            else if (v == &PL_vtbl_arylen)     s = "arylen";
+            else if (v == &PL_vtbl_glob)       s = "glob";
+            else if (v == &PL_vtbl_mglob)      s = "mglob";
+            else if (v == &PL_vtbl_nkeys)      s = "nkeys";
+            else if (v == &PL_vtbl_taint)      s = "taint";
+            else if (v == &PL_vtbl_substr)     s = "substr";
+            else if (v == &PL_vtbl_vec)        s = "vec";
+            else if (v == &PL_vtbl_pos)        s = "pos";
+            else if (v == &PL_vtbl_bm)         s = "bm";
+            else if (v == &PL_vtbl_fm)         s = "fm";
+            else if (v == &PL_vtbl_uvar)       s = "uvar";
+            else if (v == &PL_vtbl_defelem)    s = "defelem";
+#ifdef USE_LOCALE_COLLATE
+           else if (v == &PL_vtbl_collxfrm)   s = "collxfrm";
+#endif
+#ifdef OVERLOAD
+           else if (v == &PL_vtbl_amagic)     s = "amagic";
+           else if (v == &PL_vtbl_amagicelem) s = "amagicelem";
+#endif
+           if (s)
+               dump_indent(level, file, "    MG_VIRTUAL = &PL_vtbl_%s\n", s);
+           else
+               dump_indent(level, file, "    MG_VIRTUAL = 0x%lx\n", (long)v);
+        }
+       else
+           dump_indent(level, file, "    MG_VIRTUAL = 0\n");
 
-    if (!pm) {
-       dump("{}\n");
-       return;
-    }
-    dump("{\n");
-    dumplvl++;
-    if (pm->op_pmflags & PMf_ONCE)
-       ch = '?';
-    else
-       ch = '/';
-    if (pm->op_pmregexp)
-       dump("PMf_PRE %c%s%c\n",ch,pm->op_pmregexp->precomp,ch);
-    if (pm->op_type != OP_PUSHRE && pm->op_pmreplroot) {
-       dump("PMf_REPL = ");
-       dump_op(pm->op_pmreplroot);
-    }
-    if (pm->op_pmshort) {
-       dump("PMf_SHORT = %s\n",SvPEEK(pm->op_pmshort));
-    }
-    if (pm->op_pmflags) {
-       *buf = '\0';
-       if (pm->op_pmflags & PMf_USED)
-           (void)strcat(buf,"USED,");
-       if (pm->op_pmflags & PMf_ONCE)
-           (void)strcat(buf,"ONCE,");
-       if (pm->op_pmflags & PMf_SCANFIRST)
-           (void)strcat(buf,"SCANFIRST,");
-       if (pm->op_pmflags & PMf_ALL)
-           (void)strcat(buf,"ALL,");
-       if (pm->op_pmflags & PMf_SKIPWHITE)
-           (void)strcat(buf,"SKIPWHITE,");
-       if (pm->op_pmflags & PMf_CONST)
-           (void)strcat(buf,"CONST,");
-       if (pm->op_pmflags & PMf_KEEP)
-           (void)strcat(buf,"KEEP,");
-       if (pm->op_pmflags & PMf_GLOBAL)
-           (void)strcat(buf,"GLOBAL,");
-       if (pm->op_pmflags & PMf_RUNTIME)
-           (void)strcat(buf,"RUNTIME,");
-       if (pm->op_pmflags & PMf_EVAL)
-           (void)strcat(buf,"EVAL,");
-       if (*buf)
-           buf[strlen(buf)-1] = '\0';
-       dump("PMFLAGS = (%s)\n",buf);
+       if (mg->mg_private)
+           dump_indent(level, file, "    MG_PRIVATE = %d\n", mg->mg_private);
+
+       if (isPRINT(mg->mg_type))
+           dump_indent(level, file, "    MG_TYPE = '%c'\n", mg->mg_type);
+       else
+           dump_indent(level, file, "    MG_TYPE = '\\%o'\n", mg->mg_type);
+
+        if (mg->mg_flags) {
+            dump_indent(level, file, "    MG_FLAGS = 0x%02X\n", mg->mg_flags);
+           if (mg->mg_flags & MGf_TAINTEDDIR)
+               dump_indent(level, file, "      TAINTEDDIR\n");
+           if (mg->mg_flags & MGf_REFCOUNTED)
+               dump_indent(level, file, "      REFCOUNTED\n");
+            if (mg->mg_flags & MGf_GSKIP)
+               dump_indent(level, file, "      GSKIP\n");
+           if (mg->mg_flags & MGf_MINMATCH)
+               dump_indent(level, file, "      MINMATCH\n");
+        }
+       if (mg->mg_obj) {
+           dump_indent(level, file, "    MG_OBJ = 0x%lx\n", (long)mg->mg_obj);
+           if (mg->mg_flags & MGf_REFCOUNTED)
+               do_sv_dump(level+2, file, mg->mg_obj, nest+1, maxnest, dumpops, pvlim); /* MG is already +1 */
+       }
+        if (mg->mg_len)
+           dump_indent(level, file, "    MG_LEN = %d\n", mg->mg_len);
+        if (mg->mg_ptr) {
+           dump_indent(level, file, "    MG_PTR = 0x%lx", (long)mg->mg_ptr);
+           if (mg->mg_len >= 0) {
+               SV *sv = newSVpv("", 0);
+                PerlIO_printf(file, " %s", pv_display(sv, mg->mg_ptr, mg->mg_len, 0, pvlim));
+               SvREFCNT_dec(sv);
+            }
+           else if (mg->mg_len == HEf_SVKEY) {
+               PerlIO_puts(file, " => HEf_SVKEY\n");
+               do_sv_dump(level+2, file, (SV*)((mg)->mg_ptr), nest+1, maxnest, dumpops, pvlim); /* MG is already +1 */
+               continue;
+           }
+           else
+               PerlIO_puts(file, " ???? - please notify IZ");
+            PerlIO_putc(file, '\n');
+        }
     }
+}
 
-    dumplvl--;
-    dump("}\n");
+void
+magic_dump(MAGIC *mg)
+{
+    do_magic_dump(0, Perl_debug_log, mg, 0, 0, 0, 0);
 }
 
+void
+do_hv_dump(I32 level, PerlIO *file, char *name, HV *sv)
+{
+    dump_indent(level, file, "%s = 0x%lx", name, (long)sv);
+    if (sv && HvNAME(sv))
+       PerlIO_printf(file, "\t\"%s\"\n", HvNAME(sv));
+    else
+       PerlIO_putc(file, '\n');
+}
 
-#if !defined(I_STDARG) && !defined(I_VARARGS)
-/* VARARGS1 */
-static void dump(arg1,arg2,arg3,arg4,arg5)
-char *arg1;
-long arg2, arg3, arg4, arg5;
+void
+do_gv_dump(I32 level, PerlIO *file, char *name, GV *sv)
 {
-    I32 i;
+    dump_indent(level, file, "%s = 0x%lx", name, (long)sv);
+    if (sv && GvNAME(sv))
+       PerlIO_printf(file, "\t\"%s\"\n", GvNAME(sv));
+    else
+       PerlIO_putc(file, '\n');
+}
 
-    for (i = dumplvl*4; i; i--)
-       (void)PerlIO_putc(Perl_debug_log,' ');
-    PerlIO_printf(Perl_debug_log, arg1, arg2, arg3, arg4, arg5);
+void
+do_gvgv_dump(I32 level, PerlIO *file, char *name, GV *sv)
+{
+    dump_indent(level, file, "%s = 0x%lx", name, (long)sv);
+    if (sv && GvNAME(sv)) {
+       PerlIO_printf(file, "\t\"");
+       if (GvSTASH(sv) && HvNAME(GvSTASH(sv)))
+           PerlIO_printf(file, "%s\" :: \"", HvNAME(GvSTASH(sv)));
+       PerlIO_printf(file, "%s\"\n", GvNAME(sv));
+    }
+    else
+       PerlIO_putc(file, '\n');
 }
 
-#else
-
-#ifdef I_STDARG
-static void
-dump(char *pat,...)
-#else
-/*VARARGS0*/
-static void
-dump(pat,va_alist)
-    char *pat;
-    va_dcl
-#endif
+void
+do_sv_dump(I32 level, PerlIO *file, SV *sv, I32 nest, I32 maxnest, bool dumpops, STRLEN pvlim)
 {
-    I32 i;
-    va_list args;
+    dTHR;
+    SV *d = sv_newmortal();
+    char *s;
+    U32 flags;
+    U32 type;
 
-#ifdef I_STDARG
-    va_start(args, pat);
-#else
-    va_start(args);
-#endif
-    for (i = dumplvl*4; i; i--)
-       (void)PerlIO_putc(Perl_debug_log,' ');
-    PerlIO_vprintf(Perl_debug_log,pat,args);
-    va_end(args);
+    if (!sv) {
+       dump_indent(level, file, "SV = 0\n");
+       return;
+    }
+    
+    flags = SvFLAGS(sv);
+    type = SvTYPE(sv);
+
+    sv_setpvf(d, "(0x%lx) at 0x%lx\n%*s  REFCNT = %ld\n%*s  FLAGS = (",
+             (unsigned long)SvANY(sv), (unsigned long)sv,
+             PL_dumpindent*level, "", (long)SvREFCNT(sv),
+             PL_dumpindent*level, "");
+
+    if (flags & SVs_PADBUSY)   sv_catpv(d, "PADBUSY,");
+    if (flags & SVs_PADTMP)    sv_catpv(d, "PADTMP,");
+    if (flags & SVs_PADMY)     sv_catpv(d, "PADMY,");
+    if (flags & SVs_TEMP)      sv_catpv(d, "TEMP,");
+    if (flags & SVs_OBJECT)    sv_catpv(d, "OBJECT,");
+    if (flags & SVs_GMG)       sv_catpv(d, "GMG,");
+    if (flags & SVs_SMG)       sv_catpv(d, "SMG,");
+    if (flags & SVs_RMG)       sv_catpv(d, "RMG,");
+
+    if (flags & SVf_IOK)       sv_catpv(d, "IOK,");
+    if (flags & SVf_NOK)       sv_catpv(d, "NOK,");
+    if (flags & SVf_POK)       sv_catpv(d, "POK,");
+    if (flags & SVf_ROK)       sv_catpv(d, "ROK,");
+    if (flags & SVf_OOK)       sv_catpv(d, "OOK,");
+    if (flags & SVf_FAKE)      sv_catpv(d, "FAKE,");
+    if (flags & SVf_READONLY)  sv_catpv(d, "READONLY,");
+
+#ifdef OVERLOAD
+    if (flags & SVf_AMAGIC)    sv_catpv(d, "OVERLOAD,");
+#endif /* OVERLOAD */
+    if (flags & SVp_IOK)       sv_catpv(d, "pIOK,");
+    if (flags & SVp_NOK)       sv_catpv(d, "pNOK,");
+    if (flags & SVp_POK)       sv_catpv(d, "pPOK,");
+    if (flags & SVp_SCREAM)    sv_catpv(d, "SCREAM,");
+
+    switch (type) {
+    case SVt_PVCV:
+    case SVt_PVFM:
+       if (CvANON(sv))         sv_catpv(d, "ANON,");
+       if (CvUNIQUE(sv))       sv_catpv(d, "UNIQUE,");
+       if (CvCLONE(sv))        sv_catpv(d, "CLONE,");
+       if (CvCLONED(sv))       sv_catpv(d, "CLONED,");
+       if (CvNODEBUG(sv))      sv_catpv(d, "NODEBUG,");
+       break;
+    case SVt_PVHV:
+       if (HvSHAREKEYS(sv))    sv_catpv(d, "SHAREKEYS,");
+       if (HvLAZYDEL(sv))      sv_catpv(d, "LAZYDEL,");
+       break;
+    case SVt_PVGV:
+       if (GvINTRO(sv))        sv_catpv(d, "INTRO,");
+       if (GvMULTI(sv))        sv_catpv(d, "MULTI,");
+       if (GvASSUMECV(sv))     sv_catpv(d, "ASSUMECV,");
+       if (GvIMPORTED(sv)) {
+           sv_catpv(d, "IMPORT");
+           if (GvIMPORTED(sv) == GVf_IMPORTED)
+               sv_catpv(d, "ALL,");
+           else {
+               sv_catpv(d, "(");
+               if (GvIMPORTED_SV(sv))  sv_catpv(d, " SV");
+               if (GvIMPORTED_AV(sv))  sv_catpv(d, " AV");
+               if (GvIMPORTED_HV(sv))  sv_catpv(d, " HV");
+               if (GvIMPORTED_CV(sv))  sv_catpv(d, " CV");
+               sv_catpv(d, " ),");
+           }
+       }
+    case SVt_PVBM:
+       if (SvTAIL(sv))         sv_catpv(d, "TAIL,");
+       if (SvCOMPILED(sv))     sv_catpv(d, "COMPILED,");
+       break;
+    }
+
+    if (*(SvEND(d) - 1) == ',')
+       SvPVX(d)[--SvCUR(d)] = '\0';
+    sv_catpv(d, ")");
+    s = SvPVX(d);
+
+    dump_indent(level, file, "SV = ");
+    switch (type) {
+    case SVt_NULL:
+       PerlIO_printf(file, "NULL%s\n", s);
+       return;
+    case SVt_IV:
+       PerlIO_printf(file, "IV%s\n", s);
+       break;
+    case SVt_NV:
+       PerlIO_printf(file, "NV%s\n", s);
+       break;
+    case SVt_RV:
+       PerlIO_printf(file, "RV%s\n", s);
+       break;
+    case SVt_PV:
+       PerlIO_printf(file, "PV%s\n", s);
+       break;
+    case SVt_PVIV:
+       PerlIO_printf(file, "PVIV%s\n", s);
+       break;
+    case SVt_PVNV:
+       PerlIO_printf(file, "PVNV%s\n", s);
+       break;
+    case SVt_PVBM:
+       PerlIO_printf(file, "PVBM%s\n", s);
+       break;
+    case SVt_PVMG:
+       PerlIO_printf(file, "PVMG%s\n", s);
+       break;
+    case SVt_PVLV:
+       PerlIO_printf(file, "PVLV%s\n", s);
+       break;
+    case SVt_PVAV:
+       PerlIO_printf(file, "PVAV%s\n", s);
+       break;
+    case SVt_PVHV:
+       PerlIO_printf(file, "PVHV%s\n", s);
+       break;
+    case SVt_PVCV:
+       PerlIO_printf(file, "PVCV%s\n", s);
+       break;
+    case SVt_PVGV:
+       PerlIO_printf(file, "PVGV%s\n", s);
+       break;
+    case SVt_PVFM:
+       PerlIO_printf(file, "PVFM%s\n", s);
+       break;
+    case SVt_PVIO:
+       PerlIO_printf(file, "PVIO%s\n", s);
+       break;
+    default:
+       PerlIO_printf(file, "UNKNOWN(0x%x) %s\n", type, s);
+       return;
+    }
+    if (type >= SVt_PVIV || type == SVt_IV) {
+       dump_indent(level, file, "  IV = %ld", (long)SvIVX(sv));
+       if (SvOOK(sv))
+           PerlIO_printf(file, "  (OFFSET)");
+       PerlIO_putc(file, '\n');
+    }
+    if (type >= SVt_PVNV || type == SVt_NV) {
+       SET_NUMERIC_STANDARD();
+       dump_indent(level, file, "  NV = %.*g\n", DBL_DIG, SvNVX(sv));
+    }
+    if (SvROK(sv)) {
+       dump_indent(level, file, "  RV = 0x%lx\n", (long)SvRV(sv));
+       if (nest < maxnest)
+           do_sv_dump(level+1, file, SvRV(sv), nest+1, maxnest, dumpops, pvlim);
+       return;
+    }
+    if (type < SVt_PV)
+       return;
+    if (type <= SVt_PVLV) {
+       if (SvPVX(sv)) {
+           dump_indent(level, file,"  PV = 0x%lx ", (long)SvPVX(sv));
+           if (SvOOK(sv))
+               PerlIO_printf(file, "( %s . ) ", pv_display(d, SvPVX(sv)-SvIVX(sv), SvIVX(sv), 0, pvlim));
+           PerlIO_printf(file, "%s\n", pv_display(d, SvPVX(sv), SvCUR(sv), SvLEN(sv), pvlim));
+           dump_indent(level, file, "  CUR = 0\n", (long)SvCUR(sv));
+           dump_indent(level, file, "  LEN = 0\n", (long)SvLEN(sv));
+       }
+       else
+           dump_indent(level, file, "  PV = 0\n");
+    }
+    if (type >= SVt_PVMG) {
+       if (SvMAGIC(sv))
+            do_magic_dump(level, file, SvMAGIC(sv), nest, maxnest, dumpops, pvlim);
+       if (SvSTASH(sv))
+           do_hv_dump(level, file, "  STASH", SvSTASH(sv));
+    }
+    switch (type) {
+    case SVt_PVLV:
+       dump_indent(level, file, "  TYPE = %c\n", LvTYPE(sv));
+       dump_indent(level, file, "  TARGOFF = %ld\n", (long)LvTARGOFF(sv));
+       dump_indent(level, file, "  TARGLEN = %ld\n", (long)LvTARGLEN(sv));
+       dump_indent(level, file, "  TARG = 0x%lx\n", (long)LvTARG(sv));
+       /* XXX level+1 ??? */
+       do_sv_dump(level, file, LvTARG(sv), nest+1, maxnest, dumpops, pvlim);
+       break;
+    case SVt_PVAV:
+       dump_indent(level, file, "  ARRAY = 0x%lx", (long)AvARRAY(sv));
+       if (AvARRAY(sv) != AvALLOC(sv)) {
+           PerlIO_printf(file, " (offset=%d)\n", (AvARRAY(sv) - AvALLOC(sv)));
+           dump_indent(level, file, "  ALLOC = 0x%lx\n", (long)AvALLOC(sv));
+       }
+       else
+           PerlIO_putc(file, '\n');
+       dump_indent(level, file, "  FILL = %ld\n", (long)AvFILLp(sv));
+       dump_indent(level, file, "  MAX = %ld\n", (long)AvMAX(sv));
+       dump_indent(level, file, "  ARYLEN = 0x%lx\n", (long)AvARYLEN(sv));
+       flags = AvFLAGS(sv);
+       sv_setpv(d, "");
+       if (flags & AVf_REAL)   sv_catpv(d, ",REAL");
+       if (flags & AVf_REIFY)  sv_catpv(d, ",REIFY");
+       if (flags & AVf_REUSED) sv_catpv(d, ",REUSED");
+       dump_indent(level, file, "  FLAGS = (%s)\n", SvCUR(d) ? SvPVX(d) + 1 : "");
+       if (nest < maxnest && av_len((AV*)sv) >= 0) {
+           int count;
+           for (count = 0; count <=  av_len((AV*)sv) && count < maxnest; count++) {
+               SV** elt = av_fetch((AV*)sv,count,0);
+
+               dump_indent(level + 1, file, "Elt No. %ld\n", (long)count);
+               if (elt) 
+                   do_sv_dump(level+1, file, *elt, nest+1, maxnest, dumpops, pvlim);
+           }
+       }
+       break;
+    case SVt_PVHV:
+       dump_indent(level, file, "  ARRAY = 0x%lx",(long)HvARRAY(sv));
+       if (HvARRAY(sv) && HvKEYS(sv)) {
+           /* Show distribution of HEs in the ARRAY */
+           int freq[200];
+#define FREQ_MAX (sizeof freq / sizeof freq[0] - 1)
+           int i;
+           int max = 0;
+           U32 pow2 = 2, keys = HvKEYS(sv);
+           double theoret, sum = 0;
+
+           PerlIO_printf(file, "  (");
+           Zero(freq, FREQ_MAX + 1, int);
+           for (i = 0; i <= HvMAX(sv); i++) {
+               HE* h; int count = 0;
+                for (h = HvARRAY(sv)[i]; h; h = HeNEXT(h))
+                   count++;
+               if (count > FREQ_MAX)
+                   count = FREQ_MAX;
+               freq[count]++;
+               if (max < count)
+                   max = count;
+           }
+           for (i = 0; i <= max; i++) {
+               if (freq[i]) {
+                   PerlIO_printf(file, "%d%s:%d", i,
+                                 (i == FREQ_MAX) ? "+" : "",
+                                 freq[i]);
+                   if (i != max)
+                       PerlIO_printf(file, ", ");
+               }
+            }
+           PerlIO_putc(file, ')');
+           /* Now calculate quality wrt theoretical value */
+           for (i = max; i > 0; i--) { /* Precision: count down. */
+               sum += freq[i] * i * i;
+            }
+           while (keys = keys >> 1)
+               pow2 = pow2 << 1;
+           /* Approximate by Poisson distribution */
+           theoret = HvKEYS(sv);
+           theoret += theoret * theoret/pow2;
+           PerlIO_putc(file, '\n');
+           dump_indent(level, file, "  hash quality = %.1f%%", theoret/sum*100);
+       }
+       PerlIO_putc(file, '\n');
+       dump_indent(level, file, "  KEYS = %ld\n", (long)HvKEYS(sv));
+       dump_indent(level, file, "  FILL = %ld\n", (long)HvFILL(sv));
+       dump_indent(level, file, "  MAX = %ld\n", (long)HvMAX(sv));
+       dump_indent(level, file, "  RITER = %ld\n", (long)HvRITER(sv));
+       dump_indent(level, file, "  EITER = 0x%lx\n",(long) HvEITER(sv));
+       if (HvPMROOT(sv))
+           dump_indent(level, file, "  PMROOT = 0x%lx\n",(long)HvPMROOT(sv));
+       if (HvNAME(sv))
+           dump_indent(level, file, "  NAME = \"%s\"\n", HvNAME(sv));
+       if (nest < maxnest && !HvEITER(sv)) { /* Try to preserve iterator */
+           HE *he;
+           HV *hv = (HV*)sv;
+           int count = maxnest - nest;
+
+           hv_iterinit(hv);
+           while ((he = hv_iternext(hv)) && count--) {
+               SV *elt;
+               char *key;
+               I32 len;
+               U32 hash = HeHASH(he);
+
+               key = hv_iterkey(he, &len);
+               elt = hv_iterval(hv, he);
+               dump_indent(level+1, file, "Elt %s HASH = 0x%lx\n", pv_display(d, key, len, 0, pvlim), hash);
+               do_sv_dump(level+1, file, elt, nest+1, maxnest, dumpops, pvlim);
+           }
+           hv_iterinit(hv);            /* Return to status quo */
+       }
+       break;
+    case SVt_PVCV:
+       if (SvPOK(sv))
+           dump_indent(level, file, "  PROTOTYPE = \"%s\"\n", SvPV(sv,PL_na));
+       /* FALL THROUGH */
+    case SVt_PVFM:
+       do_hv_dump(level, file, "  COMP_STASH", CvSTASH(sv));
+       if (CvSTART(sv))
+           dump_indent(level, file, "  START = 0x%lx ===> %d\n", (long)CvSTART(sv), CvSTART(sv)->op_seq);
+       dump_indent(level, file, "  ROOT = 0x%lx\n", (long)CvROOT(sv));
+        if (CvROOT(sv) && dumpops)
+           do_op_dump(level+1, file, CvROOT(sv));
+       dump_indent (level, file, "  XSUB = 0x%lx\n", (long)CvXSUB(sv));
+       dump_indent (level, file, "  XSUBANY = %ld\n", (long)CvXSUBANY(sv).any_i32);
+       do_gvgv_dump(level, file, "  GVGV::GV", CvGV(sv));
+       do_gv_dump  (level, file, "  FILEGV", CvFILEGV(sv));
+       dump_indent (level, file, "  DEPTH = %ld\n", (long)CvDEPTH(sv));
+#ifdef USE_THREADS
+       dump_indent (level, file, "  MUTEXP = 0x%lx\n", (long)CvMUTEXP(sv));
+       dump_indent (level, file, "  OWNER = 0x%lx\n", (long)CvOWNER(sv));
+#endif /* USE_THREADS */
+       dump_indent (level, file, "  FLAGS = 0x%lx\n", (unsigned long)CvFLAGS(sv));
+       if (type == SVt_PVFM)
+           dump_indent(level, file, "  LINES = %ld\n", (long)FmLINES(sv));
+       dump_indent(level, file, "  PADLIST = 0x%lx\n", (long)CvPADLIST(sv));
+       if (nest < maxnest && CvPADLIST(sv)) {
+           AV* padlist = CvPADLIST(sv);
+           AV* pad_name = (AV*)*av_fetch(padlist, 0, FALSE);
+           AV* pad = (AV*)*av_fetch(padlist, 1, FALSE);
+           SV** pname = AvARRAY(pad_name);
+           SV** ppad = AvARRAY(pad);
+           I32 ix;
+
+           for (ix = 1; ix <= AvFILL(pad_name); ix++) {
+               if (SvPOK(pname[ix]))
+                   dump_indent(level, /* %5d below is enough whitespace. */
+                               file, 
+                               "%5d. 0x%lx (%s\"%s\" %ld-%ld)\n",
+                               ix, ppad[ix],
+                               SvFAKE(pname[ix]) ? "FAKE " : "",
+                               SvPVX(pname[ix]),
+                               (long)I_32(SvNVX(pname[ix])),
+                               (long)SvIVX(pname[ix]));
+           }
+       }
+       {
+           CV *outside = CvOUTSIDE(sv);
+           dump_indent(level, file, "  OUTSIDE = 0x%lx (%s)\n", 
+                       (long)outside, 
+                       (!outside ? "null"
+                        : CvANON(outside) ? "ANON"
+                        : (outside == PL_main_cv) ? "MAIN"
+                        : CvUNIQUE(outside) ? "UNIQUE"
+                        : CvGV(outside) ? GvNAME(CvGV(outside)) : "UNDEFINED"));
+       }
+       if (nest < maxnest && (CvCLONE(sv) || CvCLONED(sv)))
+           do_sv_dump(level+1, file, (SV*)CvOUTSIDE(sv), nest+1, maxnest, dumpops, pvlim);
+       break;
+    case SVt_PVGV:
+       dump_indent(level, file, "  NAME = \"%s\"\n", GvNAME(sv));
+       dump_indent(level, file, "  NAMELEN = %ld\n", (long)GvNAMELEN(sv));
+       do_hv_dump (level, file, "  GvSTASH", GvSTASH(sv));
+       dump_indent(level, file, "  GP = 0x%lx\n", (long)GvGP(sv));
+       dump_indent(level, file, "    SV = 0x%lx\n", (long)GvSV(sv));
+       dump_indent(level, file, "    REFCNT = %ld\n", (long)GvREFCNT(sv));
+       dump_indent(level, file, "    IO = 0x%lx\n", (long)GvIOp(sv));
+       dump_indent(level, file, "    FORM = 0x%lx\n", (long)GvFORM(sv));
+       dump_indent(level, file, "    AV = 0x%lx\n", (long)GvAV(sv));
+       dump_indent(level, file, "    HV = 0x%lx\n", (long)GvHV(sv));
+       dump_indent(level, file, "    CV = 0x%lx\n", (long)GvCV(sv));
+       dump_indent(level, file, "    CVGEN = 0x%lx\n", (long)GvCVGEN(sv));
+       dump_indent(level, file, "    LASTEXPR = %ld\n", (long)GvLASTEXPR(sv));
+       dump_indent(level, file, "    LINE = %ld\n", (long)GvLINE(sv));
+       dump_indent(level, file, "    FLAGS = 0x%x\n", (int)GvFLAGS(sv));
+       do_gv_dump (level, file, "    FILEGV", GvFILEGV(sv));
+       do_gv_dump (level, file, "    EGV", GvEGV(sv));
+       break;
+    case SVt_PVIO:
+       dump_indent(level, file, "  IFP = 0x%lx\n", (long)IoIFP(sv));
+       dump_indent(level, file, "  OFP = 0x%lx\n", (long)IoOFP(sv));
+       dump_indent(level, file, "  DIRP = 0x%lx\n", (long)IoDIRP(sv));
+       dump_indent(level, file, "  LINES = %ld\n", (long)IoLINES(sv));
+       dump_indent(level, file, "  PAGE = %ld\n", (long)IoPAGE(sv));
+       dump_indent(level, file, "  PAGE_LEN = %ld\n", (long)IoPAGE_LEN(sv));
+       dump_indent(level, file, "  LINES_LEFT = %ld\n", (long)IoLINES_LEFT(sv));
+       dump_indent(level, file, "  TOP_NAME = \"%s\"\n", IoTOP_NAME(sv));
+       do_gv_dump (level, file, "  TOP_GV", IoTOP_GV(sv));
+       dump_indent(level, file, "  FMT_NAME = \"%s\"\n", IoFMT_NAME(sv));
+       do_gv_dump (level, file, "  FMT_GV", IoFMT_GV(sv));
+       dump_indent(level, file, "  BOTTOM_NAME = \"%s\"\n", IoBOTTOM_NAME(sv));
+       do_gv_dump (level, file, "  BOTTOM_GV", IoBOTTOM_GV(sv));
+       dump_indent(level, file, "  SUBPROCESS = %ld\n", (long)IoSUBPROCESS(sv));
+       dump_indent(level, file, "  TYPE = %c\n", IoTYPE(sv));
+       dump_indent(level, file, "  FLAGS = 0x%lx\n", (long)IoFLAGS(sv));
+       break;
+    }
 }
-#endif
 
-#endif
+void
+sv_dump(SV *sv)
+{
+    do_sv_dump(0, Perl_debug_log, sv, 0, 0, 0, 0);
+}