Merge pull request #99 from JLHwung/fix-98

allow - in classEscape when unicode flag is enabled
jviereck · Dec 21, 2019 · 1da8afa · 1da8afa
2 parents 9bc130f + 10b8de0
commit 1da8afa
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/parser.js b/parser.js
@@ -706,6 +706,7 @@
 
       // For ClassEscape
       if (insideCharacterClass) {
+        //     b
         if (match('b')) {
           // 15.10.2.19
           // The production ClassEscape :: b evaluates by returning the
@@ -718,6 +719,10 @@
           // c ClassControlLetter
           return createEscaped('controlLetter', res[1] + 16, res[1], 2);
         }
+        //     [+U] -
+        if (match('-') && hasUnicodeFlag) {
+          return createEscaped('singleEscape', 0x002d, '\\-');
+        }
       }
 
       res = parseCharacterEscape();

diff --git a/test/test-data-unicode.json b/test/test-data-unicode.json
@@ -904,5 +904,20 @@
     "name": "SyntaxError",
     "message": "atomEscape at position 1\n    \\a\n     ^",
     "input": "\\a"
+  },
+  "[\\-]": {
+    "type": "characterClass",
+    "body": [
+      {
+        "type": "value",
+        "kind": "singleEscape",
+        "codePoint": 45,
+        "range": [1, 3],
+        "raw": "\\-"
+      }
+    ],
+    "negative": false,
+    "range": [0, 4],
+    "raw": "[\\-]"
   }
 }