diff --git a/AK/ScopeLogger.h b/AK/ScopeLogger.h index 30d766c929..d9e301d1a3 100644 --- a/AK/ScopeLogger.h +++ b/AK/ScopeLogger.h @@ -6,21 +6,21 @@ #pragma once +#include #include -#ifdef DEBUG_SPAM - namespace AK { class ScopeLogger { public: - ScopeLogger(StringView&& fun) - : m_fun(fun) +#ifdef DEBUG_SPAM + ScopeLogger(const SourceLocation& location = SourceLocation::current()) + : m_location(location) { StringBuilder sb; for (auto indent = m_depth++; indent > 0; indent--) sb.append(' '); - dbgln("\033[1;{}m{}entering {}\033[0m", m_depth % 8 + 30, sb.to_string(), m_fun); + dbgln("\033[1;{}m{}entering {}\033[0m", m_depth % 8 + 30, sb.to_string(), m_location); } ~ScopeLogger() { @@ -28,18 +28,16 @@ public: for (auto indent = --m_depth; indent > 0; indent--) sb.append(' '); - dbgln("\033[1;{}m{}leaving {}\033[0m", (m_depth + 1) % 8 + 30, sb.to_string(), m_fun); + dbgln("\033[1;{}m{}leaving {}\033[0m", (m_depth + 1) % 8 + 30, sb.to_string(), m_location); } private: static inline size_t m_depth = 0; - StringView m_fun; + SourceLocation m_location; +#else + ScopeLogger() = default; +#endif }; } using AK::ScopeLogger; -# define SCOPE_LOGGER() auto tmp##__COUNTER__ = ScopeLogger(__PRETTY_FUNCTION__); - -#else -# define SCOPE_LOGGER() -#endif diff --git a/Userland/Libraries/LibCpp/Parser.cpp b/Userland/Libraries/LibCpp/Parser.cpp index 1b945a7764..34341a61f0 100644 --- a/Userland/Libraries/LibCpp/Parser.cpp +++ b/Userland/Libraries/LibCpp/Parser.cpp @@ -54,7 +54,7 @@ void Parser::initialize_program_tokens(const StringView& program) NonnullRefPtr Parser::parse() { - SCOPE_LOGGER(); + ScopeLogger logger {}; if (m_tokens.is_empty()) return create_root_ast_node({}, {}); auto unit = create_root_ast_node(m_tokens.first().start(), m_tokens.last().end()); @@ -156,7 +156,7 @@ NonnullRefPtr Parser::parse_function_declaration(ASTNode& p NonnullRefPtr Parser::parse_function_definition(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto func = create_ast_node(parent, position(), {}); consume(Token::Type::LeftCurly); while (!eof() && peek().type() != Token::Type::RightCurly) { @@ -170,7 +170,7 @@ NonnullRefPtr Parser::parse_function_definition(ASTNode& par NonnullRefPtr Parser::parse_statement(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; ArmedScopeGuard consume_semicolon([this]() { consume(Token::Type::Semicolon); }); @@ -222,7 +222,7 @@ bool Parser::match_block_statement() NonnullRefPtr Parser::parse_block_statement(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto block_statement = create_ast_node(parent, position(), {}); consume(Token::Type::LeftCurly); while (!eof() && peek().type() != Token::Type::RightCurly) { @@ -269,7 +269,7 @@ bool Parser::match_template_arguments() NonnullRefPtrVector Parser::parse_template_arguments(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; consume(Token::Type::Less); @@ -285,7 +285,7 @@ NonnullRefPtrVector Parser::parse_template_arguments(ASTNode& parent) bool Parser::match_variable_declaration() { - SCOPE_LOGGER(); + ScopeLogger logger {}; save_state(); ScopeGuard state_guard = [this] { load_state(); }; @@ -316,7 +316,7 @@ bool Parser::match_variable_declaration() NonnullRefPtr Parser::parse_variable_declaration(ASTNode& parent, bool expect_semicolon) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto var = create_ast_node(parent, position(), {}); if (!match_variable_declaration()) { error("unexpected token for variable type"); @@ -344,7 +344,7 @@ NonnullRefPtr Parser::parse_variable_declaration(ASTNode& p NonnullRefPtr Parser::parse_expression(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto expression = parse_primary_expression(parent); // TODO: remove eof() logic, should still work without it if (eof() || match(Token::Type::Semicolon)) { @@ -406,7 +406,7 @@ bool Parser::match_secondary_expression() NonnullRefPtr Parser::parse_primary_expression(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; // TODO: remove eof() logic, should still work without it if (eof()) { auto node = create_ast_node(parent, position(), position()); @@ -536,7 +536,7 @@ NonnullRefPtr Parser::parse_literal(ASTNode& parent) NonnullRefPtr Parser::parse_secondary_expression(ASTNode& parent, NonnullRefPtr lhs) { - SCOPE_LOGGER(); + ScopeLogger logger {}; switch (peek().type()) { case Token::Type::Plus: return parse_binary_expression(parent, lhs, BinaryOp::Addition); @@ -689,7 +689,7 @@ bool Parser::match_function_declaration() Optional> Parser::parse_parameter_list(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; NonnullRefPtrVector parameters; while (peek().type() != Token::Type::RightParen && !eof()) { if (match_ellipsis()) { @@ -739,7 +739,7 @@ bool Parser::match_preprocessor() void Parser::consume_preprocessor() { - SCOPE_LOGGER(); + ScopeLogger logger {}; switch (peek().type()) { case Token::Type::PreprocessorStatement: consume(); @@ -756,7 +756,7 @@ void Parser::consume_preprocessor() Optional Parser::consume_whitespace() { - SCOPE_LOGGER(); + ScopeLogger logger {}; return consume(Token::Type::Whitespace); } @@ -832,7 +832,7 @@ String Parser::text_in_range(Position start, Position end) const void Parser::error(StringView message) { - SCOPE_LOGGER(); + ScopeLogger logger {}; if (message.is_null() || message.is_empty()) message = ""; String formatted_message; @@ -937,7 +937,7 @@ void Parser::print_tokens() const NonnullRefPtr Parser::parse_string_literal(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; Optional start_token_index; Optional end_token_index; while (!eof()) { @@ -971,7 +971,7 @@ NonnullRefPtr Parser::parse_string_literal(ASTNode& parent) NonnullRefPtr Parser::parse_return_statement(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto return_statement = create_ast_node(parent, position(), {}); consume(Token::Type::Keyword); if (!peek(Token::Type::Semicolon).has_value()) { @@ -984,7 +984,7 @@ NonnullRefPtr Parser::parse_return_statement(ASTNode& parent) NonnullRefPtr Parser::parse_enum_declaration(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto enum_decl = create_ast_node(parent, position(), {}); consume_keyword("enum"); auto name_token = consume(Token::Type::Identifier); @@ -1031,7 +1031,7 @@ bool Parser::match_keyword(const String& keyword) NonnullRefPtr Parser::parse_struct_or_class_declaration(ASTNode& parent, StructOrClassDeclaration::Type type) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto decl = create_ast_node(parent, position(), {}, type); switch (type) { case StructOrClassDeclaration::Type::Struct: @@ -1058,7 +1058,7 @@ NonnullRefPtr Parser::parse_struct_or_class_declaratio NonnullRefPtr Parser::parse_member_declaration(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto member_decl = create_ast_node(parent, position(), {}); member_decl->m_type = parse_type(*member_decl); @@ -1077,7 +1077,7 @@ NonnullRefPtr Parser::parse_member_declaration(ASTNode& paren NonnullRefPtr Parser::parse_boolean_literal(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto token = consume(Token::Type::Keyword); auto text = text_of_token(token); // text == "true" || text == "false"; @@ -1096,7 +1096,7 @@ bool Parser::match_boolean_literal() NonnullRefPtr Parser::parse_type(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; if (!match_type()) { auto token = consume(); @@ -1134,7 +1134,7 @@ NonnullRefPtr Parser::parse_type(ASTNode& parent) NonnullRefPtr Parser::parse_for_statement(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto for_statement = create_ast_node(parent, position(), {}); consume(Token::Type::Keyword); consume(Token::Type::LeftParen); @@ -1158,7 +1158,7 @@ NonnullRefPtr Parser::parse_for_statement(ASTNode& parent) NonnullRefPtr Parser::parse_if_statement(ASTNode& parent) { - SCOPE_LOGGER(); + ScopeLogger logger {}; auto if_statement = create_ast_node(parent, position(), {}); consume(Token::Type::Keyword); consume(Token::Type::LeftParen); @@ -1176,7 +1176,7 @@ NonnullRefPtr Parser::parse_if_statement(ASTNode& parent) } Vector Parser::parse_type_qualifiers() { - SCOPE_LOGGER(); + ScopeLogger logger {}; Vector qualifiers; while (!eof()) { auto token = peek(); @@ -1195,7 +1195,7 @@ Vector Parser::parse_type_qualifiers() Vector Parser::parse_function_qualifiers() { - SCOPE_LOGGER(); + ScopeLogger logger {}; Vector qualifiers; while (!eof()) { auto token = peek();