mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-07-29 20:29:18 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibJS: Actually pop frames off of the scope stack when exiting a scope

Browse source
This commit is contained in:
Andreas Kling 2020-03-27 11:34:58 +01:00
commit 9494865f99
Notes: sideshowbarker 2024-07-19 08:06:42 +09:00
1 changed files with 5 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Libraries/LibJS/Interpreter.cpp
View file

@ -86,8 +86,11 @@ void Interpreter::enter_scope(const ScopeNode& scope_node, Vector<Argument> argu
void Interpreter::exit_scope(const ScopeNode& scope_node) void Interpreter::exit_scope(const ScopeNode& scope_node)
{ {
while (m_scope_stack.last().scope_node.ptr() != &scope_node) while (!m_scope_stack.is_empty()) {
m_scope_stack.take_last(); auto popped_scope = m_scope_stack.take_last();
if (popped_scope.scope_node.ptr() == &scope_node)
break;
}
// If we unwind all the way, just reset m_unwind_until so that future "return" doesn't break. // If we unwind all the way, just reset m_unwind_until so that future "return" doesn't break.
if (m_scope_stack.is_empty()) if (m_scope_stack.is_empty())