Skip to content

Commit 6d38f15

Browse files
committed
Perl_dump_sub_perl: call S_do_op_dump_bar directly
A minor refactor in preparation for the next commit: make Perl_dump_sub_perl() invoke S_do_op_dump_bar() directly, rather than going via op_dump() which indirectly calls the former. Should make no functional difference.
1 parent 35482d4 commit 6d38f15

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

dump.c

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -873,6 +873,12 @@ Perl_dump_sub(pTHX_ const GV *gv)
873873
dump_sub_perl(gv, FALSE);
874874
}
875875

876+
877+
/* forward decl */
878+
static void
879+
S_do_op_dump_bar(pTHX_ I32 level, UV bar, PerlIO *file, const OP *o);
880+
881+
876882
void
877883
Perl_dump_sub_perl(pTHX_ const GV *gv, bool justperl)
878884
{
@@ -901,7 +907,7 @@ Perl_dump_sub_perl(pTHX_ const GV *gv, bool justperl)
901907
PTR2UV(CvXSUB(cv)),
902908
(int)CvXSUBANY(cv).any_i32);
903909
else if (CvROOT(cv))
904-
op_dump(CvROOT(cv));
910+
S_do_op_dump_bar(aTHX_ 0, 0, Perl_debug_log, CvROOT(cv));
905911
else
906912
Perl_dump_indent(aTHX_ 0, Perl_debug_log, "<undef>\n");
907913
}
@@ -966,11 +972,6 @@ S_gv_display(pTHX_ GV *gv)
966972

967973

968974

969-
/* forward decl */
970-
static void
971-
S_do_op_dump_bar(pTHX_ I32 level, UV bar, PerlIO *file, const OP *o);
972-
973-
974975
static void
975976
S_do_pmop_dump_bar(pTHX_ I32 level, UV bar, PerlIO *file, const PMOP *pm)
976977
{

0 commit comments

Comments
 (0)