Skip to content

Commit 5895b96

Browse files
authored
Class extends expression should be parsed is strict mode (#3801)
JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik frobert@inf.u-szeged.hu
1 parent e76a138 commit 5895b96

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

jerry-core/parser/js/js-parser-expr.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -798,6 +798,11 @@ parser_parse_class (parser_context_t *context_p, /**< context */
798798
parser_emit_cbc_ext (context_p, CBC_EXT_PUSH_ANONYMOUS_CLASS_ENV);
799799
}
800800

801+
bool is_strict = (context_p->status_flags & PARSER_IS_STRICT) != 0;
802+
803+
/* 14.5. A ClassBody is always strict code. */
804+
context_p->status_flags |= PARSER_IS_STRICT;
805+
801806
if (context_p->token.type == LEXER_KEYW_EXTENDS)
802807
{
803808
lexer_next_token (context_p);
@@ -815,11 +820,6 @@ parser_parse_class (parser_context_t *context_p, /**< context */
815820
parser_raise_error (context_p, PARSER_ERR_LEFT_BRACE_EXPECTED);
816821
}
817822

818-
bool is_strict = context_p->status_flags & PARSER_IS_STRICT;
819-
820-
/* 14.5. A ClassBody is always strict code. */
821-
context_p->status_flags |= PARSER_IS_STRICT;
822-
823823
/* ClassDeclaration is parsed. Continue with class body. */
824824
parser_parse_class_literal (context_p, opts);
825825

tests/test262-es6-excludelist.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,6 @@
517517
<test id="language/statements/class/name-binding/in-extends-expression-assigned.js"><reason></reason></test>
518518
<test id="language/statements/class/name-binding/in-extends-expression.js"><reason></reason></test>
519519
<test id="language/statements/class/strict-mode/arguments-caller.js"><reason></reason></test>
520-
<test id="language/statements/class/strict-mode/with.js"><reason></reason></test>
521520
<test id="language/statements/class/syntax/early-errors/class-body-static-method-get-propname-prototype.js"><reason></reason></test>
522521
<test id="language/statements/continue/labeled-continue.js"><reason></reason></test>
523522
<test id="language/statements/continue/nested-let-bound-for-loops-labeled-continue.js"><reason></reason></test>

0 commit comments

Comments
 (0)