From db422fa49977ce7fdf08126488dc54e3393b3a43 Mon Sep 17 00:00:00 2001 From: Ben Wiederhake Date: Sat, 29 Aug 2020 13:25:37 +0200 Subject: [PATCH] LibJS: Avoid unnecessary lambda Especially when it's evaluated immediately and unconditionally. --- Libraries/LibJS/Parser.cpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/Libraries/LibJS/Parser.cpp b/Libraries/LibJS/Parser.cpp index 275c06eaab..c8b70a874e 100644 --- a/Libraries/LibJS/Parser.cpp +++ b/Libraries/LibJS/Parser.cpp @@ -263,10 +263,9 @@ NonnullRefPtr Parser::parse_program() NonnullRefPtr Parser::parse_statement() { - auto statement = [this]() -> NonnullRefPtr { switch (m_parser_state.m_current_token.type()) { case TokenType::Class: - return parse_class_declaration(); + return parse_class_declaration(); case TokenType::Function: { auto declaration = parse_function_node(); m_parser_state.m_function_scopes.last().append(declaration); @@ -291,7 +290,7 @@ NonnullRefPtr Parser::parse_statement() case TokenType::Break: return parse_break_statement(); case TokenType::Continue: - return parse_continue_statement(); + return parse_continue_statement(); case TokenType::Switch: return parse_switch_statement(); case TokenType::Do: @@ -299,7 +298,7 @@ NonnullRefPtr Parser::parse_statement() case TokenType::While: return parse_while_statement(); case TokenType::Debugger: - return parse_debugger_statement(); + return parse_debugger_statement(); case TokenType::Semicolon: consume(); return create_ast_node(); @@ -317,9 +316,7 @@ NonnullRefPtr Parser::parse_statement() expected("statement (missing switch case)"); consume(); return create_ast_node(); - } }(); - - return statement; + } } RefPtr Parser::try_parse_arrow_function_expression(bool expect_parens)