summaryrefslogtreecommitdiff
path: root/dump.c
diff options
context:
space:
mode:
authorDavid Mitchell <davem@iabyn.com>2016-11-12 08:05:53 +0000
committerDavid Mitchell <davem@iabyn.com>2016-11-12 16:15:09 +0000
commit8efda520355126b24fb7c81c753eb2028dcc43bd (patch)
treee9a95c1e6d605973f58fadd365cb9c017af2c2c0 /dump.c
parente97ca23069de91e6fba421ba7674397837455073 (diff)
downloadperl-8efda520355126b24fb7c81c753eb2028dcc43bd.tar.gz
op_dump() - remove extra indentation from PMOP
When dumping a PMOP, it displays the PMOP-specific fields with an extra set of braces and level of indentation, e.g. { TYPE = match ===> 1 FLAGS = (VOID,SLABBED) PRIVATE = (RTIME) { PMf_PRE /abc/ (RUNTIME) PMFLAGS = (SCANFIRST,ALL) } } This is visually confusing, because child ops are shown in the same way. This commit removes the extra indentation: { TYPE = match ===> 1 FLAGS = (VOID,SLABBED) PRIVATE = (RTIME) PMf_PRE /abc/ (RUNTIME) PMFLAGS = (SCANFIRST,ALL) }
Diffstat (limited to 'dump.c')
-rw-r--r--dump.c8
1 files changed, 1 insertions, 7 deletions
diff --git a/dump.c b/dump.c
index 58c8fccb9b..be89333971 100644
--- a/dump.c
+++ b/dump.c
@@ -657,12 +657,8 @@ Perl_do_pmop_dump(pTHX_ I32 level, PerlIO *file, const PMOP *pm)
PERL_ARGS_ASSERT_DO_PMOP_DUMP;
- if (!pm) {
- Perl_dump_indent(aTHX_ level, file, "{}\n");
+ if (!pm)
return;
- }
- Perl_dump_indent(aTHX_ level, file, "{\n");
- level++;
if (pm->op_pmflags & PMf_ONCE)
ch = '?';
else
@@ -698,8 +694,6 @@ Perl_do_pmop_dump(pTHX_ I32 level, PerlIO *file, const PMOP *pm)
Perl_dump_indent(aTHX_ level, file, "PMFLAGS = (%s)\n", SvCUR(tmpsv) ? SvPVX_const(tmpsv) + 1 : "");
SvREFCNT_dec_NN(tmpsv);
}
-
- Perl_dump_indent(aTHX_ level-1, file, "}\n");
}
const struct flag_to_name pmflags_flags_names[] = {