mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-09-23 17:58:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibJS: Don't extend arguments object to match the parameter count

Browse source 
The `arguments` object should only have the *arguments* as numeric
properties, not the *parameters*.

Given this function:

    function foo(a, b) {
        return arguments.length;
    }

Calling foo() with no arguments now correctly returns 0 instead of 2.
This commit is contained in:
Andreas Kling 2021-06-27 00:37:07 +02:00
commit f61a9f2dc5
Notes: sideshowbarker 2024-07-18 11:28:19 +09:00
1 changed files with 0 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
Userland/Libraries/LibJS/Runtime/ScriptFunction.cpp
View file

@ -170,9 +170,6 @@ Value ScriptFunction::execute_function_body()
argument_value = js_undefined(); argument_value = js_undefined();
} }
if (i >= execution_context_arguments.size())
execution_context_arguments.resize(i + 1);
execution_context_arguments[i] = argument_value;
vm.assign(param, argument_value, global_object(), true, vm.lexical_environment()); vm.assign(param, argument_value, global_object(), true, vm.lexical_environment());
}); });