Merge pull request #76 from jviereck/fix-stuck-bail

Adding bail error output in case the parser gets stuck
jviereck · Aug 16, 2015 · 24c38cd · 24c38cd
2 parents 0411839 + b38d294
commit 24c38cd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/parser.js b/parser.js
@@ -926,7 +926,7 @@
     var result = parseDisjunction();
 
     if (result.range[1] !== str.length) {
-      throw SyntaxError('Could not parse entire input - got stuck: ' + str);
+      bail('Could not parse entire input - got stuck', '', result.range[1]);
     }
 
     // The spec requires to interpret the `\2` in `/\2()()/` as backreference.

diff --git a/test/test-data.json b/test/test-data.json
@@ -4712,7 +4712,7 @@
   ")*": {
     "type": "error",
     "name": "SyntaxError",
-    "message": "Could not parse entire input - got stuck: )*",
+    "message": "Could not parse entire input - got stuck at position 0\n    )*\n    ^",
     "input": ")*"
   },
   "**a": {
@@ -30228,7 +30228,7 @@
   "\\": {
     "type": "error",
     "name": "SyntaxError",
-    "message": "Could not parse entire input - got stuck: \\",
+    "message": "Could not parse entire input - got stuck at position 2\n    \\\n      ^",
     "input": "\\"
   },
   "\\!": {