From d4bfcea570acab122f1c4048962479b0d02a7370 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Mon, 4 May 2020 19:01:24 +0100 Subject: [PATCH] LibJS: Add indentation to sections in SwitchCase::dump() This now matches the output of Program (Variables) ... (Children) ... or FunctionDeclaration 'foo' (Parameters) ... (Body) ... etc. Also don't print each consequent statement index, it doesn't add any value. --- Libraries/LibJS/AST.cpp | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/Libraries/LibJS/AST.cpp b/Libraries/LibJS/AST.cpp index a53eac4e001..0544b23321c 100644 --- a/Libraries/LibJS/AST.cpp +++ b/Libraries/LibJS/AST.cpp @@ -1384,21 +1384,17 @@ void SwitchStatement::dump(int indent) const void SwitchCase::dump(int indent) const { ASTNode::dump(indent); - print_indent(indent); + print_indent(indent + 1); if (m_test) { printf("(Test)\n"); - m_test->dump(indent + 1); + m_test->dump(indent + 2); } else { printf("(Default)\n"); } - print_indent(indent); + print_indent(indent + 1); printf("(Consequent)\n"); - int i = 0; - for (auto& statement : m_consequent) { - print_indent(indent); - printf("[%d]\n", i++); - statement.dump(indent + 1); - } + for (auto& statement : m_consequent) + statement.dump(indent + 2); } Value ConditionalExpression::execute(Interpreter& interpreter) const