Skip to content

Commit 75e48b7

Browse files
authored
[[FIX]] change escape-sequence handler for double quotes (\") (#3566)
* [[FIX]] change escape-sequence handler for double quotes * remove the 'case' for backslash escape-sequence handling * add a unit test for backslash escape-sequence handling * rename test
1 parent 4a681b9 commit 75e48b7

2 files changed

Lines changed: 11 additions & 3 deletions

File tree

src/lex.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1114,9 +1114,6 @@ Lexer.prototype = {
11141114
case "\\":
11151115
char = "\\\\";
11161116
break;
1117-
case "\"":
1118-
char = "\\\"";
1119-
break;
11201117
case "/":
11211118
break;
11221119
case "":

tests/unit/core.js

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2312,6 +2312,17 @@ exports.duplicateProto = function (test) {
23122312
//.addError(3, 23232323, "Duplicate key '__proto__'.")
23132313
.test(src, { proto: true });
23142314

2315+
src = [
2316+
"void {",
2317+
" '\"': null,",
2318+
" \"\\\"\": null",
2319+
"};"
2320+
];
2321+
2322+
TestRun(test, "Duplicate keys (backslash)")
2323+
.addError(3, 7, "Duplicate key '\"'.")
2324+
.test(src);
2325+
23152326
src = [
23162327
"void {",
23172328
" __proto__: null,",

0 commit comments

Comments
 (0)