Class extends expression should be parsed is strict mode (#3801)

JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik frobert@inf.u-szeged.hu
jerryscript-project · May 28, 2020 · 5895b96 · 5895b96
1 parent e76a138
commit 5895b96
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/jerry-core/parser/js/js-parser-expr.c b/jerry-core/parser/js/js-parser-expr.c
@@ -798,6 +798,11 @@ parser_parse_class (parser_context_t *context_p, /**< context */
     parser_emit_cbc_ext (context_p, CBC_EXT_PUSH_ANONYMOUS_CLASS_ENV);
   }
 
+  bool is_strict = (context_p->status_flags & PARSER_IS_STRICT) != 0;
+
+  /* 14.5. A ClassBody is always strict code. */
+  context_p->status_flags |= PARSER_IS_STRICT;
+
   if (context_p->token.type == LEXER_KEYW_EXTENDS)
   {
     lexer_next_token (context_p);
@@ -815,11 +820,6 @@ parser_parse_class (parser_context_t *context_p, /**< context */
     parser_raise_error (context_p, PARSER_ERR_LEFT_BRACE_EXPECTED);
   }
 
-  bool is_strict = context_p->status_flags & PARSER_IS_STRICT;
-
-  /* 14.5. A ClassBody is always strict code. */
-  context_p->status_flags |= PARSER_IS_STRICT;
-
   /* ClassDeclaration is parsed. Continue with class body. */
   parser_parse_class_literal (context_p, opts);
 

diff --git a/tests/test262-es6-excludelist.xml b/tests/test262-es6-excludelist.xml
@@ -517,7 +517,6 @@
   <test id="language/statements/class/name-binding/in-extends-expression-assigned.js"><reason></reason></test>
   <test id="language/statements/class/name-binding/in-extends-expression.js"><reason></reason></test>
   <test id="language/statements/class/strict-mode/arguments-caller.js"><reason></reason></test>
-  <test id="language/statements/class/strict-mode/with.js"><reason></reason></test>
   <test id="language/statements/class/syntax/early-errors/class-body-static-method-get-propname-prototype.js"><reason></reason></test>
   <test id="language/statements/continue/labeled-continue.js"><reason></reason></test>
   <test id="language/statements/continue/nested-let-bound-for-loops-labeled-continue.js"><reason></reason></test>