archive-gh-me/jerryscript
1
0
Fork 0
mirror of https://github.com/jerryscript-project/jerryscript.git synced 2025-12-15 16:29:21 +00:00
Code Issues Projects Releases Wiki Activity

Identifiers should be stored in the lexical environment if argument object needs to be created (#2816)

Browse Source 
This patch fixes #2699.

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik frobert@inf.u-szeged.hu
This commit is contained in:
Robert Fancsik 2019-04-12 13:48:02 +02:00 committed by Dániel Bátyai
parent 40f7b1c27f
commit 48cec485e1
2 changed files with 26 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
jerry-core/parser/js/js-parser.c
View File

@ -226,7 +226,7 @@ parser_compute_indicies (parser_context_t *context_p, /**< context */
{
if (literal_p->status_flags & LEXER_FLAG_VAR)
{
if (status_flags & PARSER_NO_REG_STORE)
if (status_flags & (PARSER_NO_REG_STORE | PARSER_ARGUMENTS_NEEDED))
{
literal_p->status_flags |= LEXER_FLAG_NO_REG_STORE;
}

25
tests/jerry/regression-test-issue-2699.js Normal file
View File

@ -0,0 +1,25 @@
// Copyright JS Foundation and other contributors, http://js.foundation
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
function f(a) {
function a() {
return 5;
}
assert (typeof a === "function");
assert (a () === 5);
assert (arguments[0] () === 5);
}
f (6);