Skip to content

Commit 799583e

Browse files
author
Robert Fancsik
authored
Private identifier resolving should always construct new literal object (#4951)
This patch fixes #4930. JerryScript-DCO-1.0-Signed-off-by: Robert Fancsik [email protected]
1 parent 2a37821 commit 799583e

File tree

2 files changed

+31
-0
lines changed

2 files changed

+31
-0
lines changed

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1517,6 +1517,7 @@ parser_resolve_private_identifier (parser_context_t *context_p) /**< context */
15171517

15181518
if (!(context_iter_p->opts & SCANNER_SUCCESSFUL_CLASS_SCAN))
15191519
{
1520+
lexer_construct_literal_object (context_p, &context_p->token.lit_location, LEXER_STRING_LITERAL);
15201521
return;
15211522
}
15221523

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
// Copyright JS Foundation and other contributors, http://js.foundation
2+
//
3+
// Licensed under the Apache License, Version 2.0 (the "License");
4+
// you may not use this file except in compliance with the License.
5+
// You may obtain a copy of the License at
6+
//
7+
// http://www.apache.org/licenses/LICENSE-2.0
8+
//
9+
// Unless required by applicable law or agreed to in writing, software
10+
// distributed under the License is distributed on an "AS IS" BASIS
11+
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12+
// See the License for the specific language governing permissions and
13+
// limitations under the License.
14+
15+
try {
16+
eval(`class JSEtest {
17+
#test262;
18+
19+
getPrivateReference() {
20+
return this.#test262;
21+
}
22+
23+
setPrivateReference(value) {function this.#test262 = value;
24+
}
25+
};
26+
`);
27+
assert(false);
28+
} catch (e) {
29+
assert(e instanceof SyntaxError);
30+
}

0 commit comments

Comments
 (0)