[[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
jshint · Sep 12, 2021 · 75e48b7 · 75e48b7
1 parent 4a681b9
commit 75e48b7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/src/lex.js b/src/lex.js
@@ -1114,9 +1114,6 @@ Lexer.prototype = {
     case "\\":
       char = "\\\\";
       break;
-    case "\"":
-      char = "\\\"";
-      break;
     case "/":
       break;
     case "":

diff --git a/tests/unit/core.js b/tests/unit/core.js
@@ -2312,6 +2312,17 @@ exports.duplicateProto = function (test) {
     //.addError(3, 23232323, "Duplicate key '__proto__'.")
     .test(src, { proto: true });
 
+  src = [
+    "void {",
+    "  '\"': null,",
+    "  \"\\\"\": null",
+    "};"
+  ];
+
+  TestRun(test, "Duplicate keys (backslash)")
+    .addError(3, 7, "Duplicate key '\"'.")
+    .test(src);
+
   src = [
     "void {",
     "  __proto__: null,",