YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix error for duplicated names in ArrowFunction paremeter list (#3597)

Browse Source 
Duplicated parameter names are not allowed for arrow functions
regardless of strict mode.

JerryScript-DCO-1.0-Signed-off-by: Rafal Walczyna r.walczyna@samsung.com
This commit is contained in:
Rafal Walczyna
2020-03-03 09:18:27 +01:00
committed by GitHub
parent 34edf53753
commit 3c8a6ca71d
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-core/parser/js/js-parser.c
+2 -1
View File
@@ -1809,7 +1809,8 @@ parser_parse_function_arguments (parser_context_t *context_p, /**< context */
if (JERRY_UNLIKELY (context_p->lit_object.literal_p->status_flags & LEXER_FLAG_FUNCTION_ARGUMENT)) if (JERRY_UNLIKELY (context_p->lit_object.literal_p->status_flags & LEXER_FLAG_FUNCTION_ARGUMENT))
{ {
#if ENABLED (JERRY_ES2015) #if ENABLED (JERRY_ES2015)
if (context_p->status_flags & PARSER_FUNCTION_HAS_NON_SIMPLE_PARAM) if ((context_p->status_flags & PARSER_FUNCTION_HAS_NON_SIMPLE_PARAM)
|| (context_p->status_flags & PARSER_IS_ARROW_FUNCTION))
{ {
parser_raise_error (context_p, PARSER_ERR_DUPLICATED_ARGUMENT_NAMES); parser_raise_error (context_p, PARSER_ERR_DUPLICATED_ARGUMENT_NAMES);
} }
tests/jerry/es2015/arrow-function.js
+2
View File
@@ -154,6 +154,8 @@ must_throw ("!x => 4");
must_throw ("x => {} = 1"); must_throw ("x => {} = 1");
must_throw ("x => {} a = 1"); must_throw ("x => {} a = 1");
must_throw ("x => {} ? 1 : 0"); must_throw ("x => {} ? 1 : 0");
must_throw ("(x,x,x) => 0");
must_throw ("(x,x,x) => { }");
must_throw_strict ("(package) => 0"); must_throw_strict ("(package) => 0");
must_throw_strict ("(package) => { return 5 }"); must_throw_strict ("(package) => { return 5 }");
must_throw_strict ("(x,x,x) => 0"); must_throw_strict ("(x,x,x) => 0");