RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 15:07:45 +00:00
Code Issues Activity Actions

Revert "LibJS: Don't hoist functions under certain circumstances"

Browse source 
This reverts commit 3411d50737.

It was causing LeakSanitizer on CI to fail, possibly due to a circular
reference.
This commit is contained in:
Linus Groh 2021-07-06 12:21:12 +01:00
parent 73183ee5c4
commit 3faeabf1dc
4 changed files with 8 additions and 45 deletions
Download patch file Download diff file Expand all files Collapse all files

39
Userland/Libraries/LibJS/Parser.cpp
View file

@ -64,24 +64,7 @@ public:
scope_node->add_variables(m_parser.m_state.let_scopes.last());
scope_node->add_functions(m_parser.m_state.current_scope->function_declarations);
for (auto& hoistable_function : m_parser.m_state.current_scope->hoisted_function_declarations) {
if (is_hoistable(hoistable_function)) {
scope_node->add_hoisted_function(hoistable_function.declaration);
}
}
}
static bool is_hoistable(Parser::Scope::HoistableDeclaration& declaration)
{
auto& name = declaration.declaration->name();
// See if we find any conflicting lexical declaration on the way up
for (RefPtr<Parser::Scope> scope = declaration.scope; !scope.is_null(); scope = scope->parent) {
if (scope->lexical_declarations.contains(name)) {
return false;
}
}
return true;
scope_node->add_hoisted_functions(m_parser.m_state.current_scope->hoisted_function_declarations);
}
Parser& m_parser;
@ -321,8 +304,7 @@ NonnullRefPtr<Declaration> Parser::parse_declaration()
case TokenType::Function: {
auto declaration = parse_function_node<FunctionDeclaration>();
m_state.current_scope->function_declarations.append(declaration);
auto hoisting_target = m_state.current_scope->get_current_function_scope();
hoisting_target->hoisted_function_declarations.append({ declaration, *m_state.current_scope });
m_state.current_scope->get_current_function_scope()->hoisted_function_declarations.append(declaration);
return declaration;
}
case TokenType::Let:
@ -1778,23 +1760,10 @@ NonnullRefPtr<VariableDeclaration> Parser::parse_variable_declaration(bool for_l
consume_or_insert_semicolon();
auto declaration = create_ast_node<VariableDeclaration>({ m_state.current_token.filename(), rule_start.position(), position() }, declaration_kind, move(declarations));
if (declaration_kind == DeclarationKind::Var) {
if (declaration_kind == DeclarationKind::Var)
m_state.var_scopes.last().append(declaration);
} else {
else
m_state.let_scopes.last().append(declaration);
for (auto& declarator : declaration->declarations()) {
declarator.target().visit(
[&](const NonnullRefPtr<Identifier>& id) {
m_state.current_scope->lexical_declarations.set(id->string());
},
[&](const NonnullRefPtr<BindingPattern>& binding) {
binding->for_each_bound_name([&](const auto& name) {
m_state.current_scope->lexical_declarations.set(name);
});
});
}
}
return declaration;
}