diff options
author | David Mitchell <davem@iabyn.com> | 2016-11-12 08:05:53 +0000 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2016-11-12 16:15:09 +0000 |
commit | 8efda520355126b24fb7c81c753eb2028dcc43bd (patch) | |
tree | e9a95c1e6d605973f58fadd365cb9c017af2c2c0 /dump.c | |
parent | e97ca23069de91e6fba421ba7674397837455073 (diff) | |
download | perl-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.c | 8 |
1 files changed, 1 insertions, 7 deletions
@@ -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[] = { |