Merge pull request #134 from benvan/backslash-bug

Bug fix: quotes incorrectly parsed by wrapNumbers resulting in broken json

Author: tulios

extension/src/json-viewer/content-extractor.js

@@ -71,14 +71,15 @@ function wrapNumbers(text) {
   var buffer = "";
   var numberBuffer = "";
   var isInString = false;
+  var charIsEscaped = false;
   var isInNumber = false;
   var previous = "";
   var beforePrevious = "";
 
   for (var i = 0, len = text.length; i < len; i++) {
     var char = text[i];
 
-    if (char == '"' && (previous != '\\' || (previous == '\\' && beforePrevious == '\\'))) {
+    if (char == '"' && !charIsEscaped) {
       isInString = !isInString;
     }
 
@@ -99,6 +100,9 @@ function wrapNumbers(text) {
       numberBuffer = "";
     }
 
+    // this applies to the _next_ character - the one used in the next iteration
+    charIsEscaped = (char == '\\') ? !charIsEscaped : false
+
     if (isInNumber) {
       numberBuffer += char;
 

tests/test.backslash.json

@@ -1,4 +1,5 @@
 {
+  "triple-backslash": "\\\"99\\\"",
   "path": "C:\\foobar\\",
   "number": 123,
   "foo": "bar"