fix bug: pick from extended set last

KilledByAPixel · Dec 10, 2019 · 40b7949 · 40b7949
1 parent 2e8aaec
commit 40b7949
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/JSONCrush.js b/JSONCrush.js
@@ -109,14 +109,6 @@ function JSONCrush(string)
     // create a string of replacement characters
     let characters = [];
 
-    // pick from extended set last
-    for (let i=255; i>32; --i)
-    {
-        let c = String.fromCharCode(i);
-        if (c!='\\' && !characters.includes(c))
-            characters.push(c);
-    }
-
     // prefer replacing with characters that will not be escaped by encodeURIComponent
     const unescapedCharacters = `-_.!~*'()`;
     for (let i=127; --i;)
@@ -130,6 +122,14 @@ function JSONCrush(string)
         )
             characters.push(String.fromCharCode(i));
     }
+
+    // pick from extended set last
+    for (let i=32; i<255; ++i)
+    {
+        let c = String.fromCharCode(i);
+        if (c!='\\' && !characters.includes(c))
+            characters.unshift(c);
+    }
 
     // remove delimiter if it is found in the string
     string = string.replace(new RegExp(delimiter,'g'),'');

diff --git a/JSONCrush.min.js b/JSONCrush.min.js
diff --git a/index.html b/index.html
@@ -42,7 +42,7 @@
 <br>
 <br>
 <br><div id=div_unit_test_results></div>
-<script src="JSONCrush.js?9"></script>
+<script src="JSONCrush.js?10"></script>
 <script>
 
 "use strict"; // strict mode