Optimize: Use new String to avoid unnecessary string object creation

bestiejs · Dec 24, 2014 · cefa14a · cefa14a
1 parent 9932ca4
commit cefa14a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/json3.js b/lib/json3.js
@@ -74,7 +74,7 @@
       if (name == "bug-string-char-index") {
         // IE <= 7 doesn't support accessing string characters using square
         // bracket notation. IE 8 only supports this for primitives.
-        isSupported = "a"[0] != "a";
+        isSupported = ("a"[0] + Object("a")[0]) != "aa";
       } else if (name == "json") {
         // Indicates whether both `JSON.stringify` and `JSON.parse` are
         // supported.
@@ -871,7 +871,7 @@
         exports.parse = function (source, callback) {
           var result, value;
           Index = 0;
-          Source = "" + source;
+          Source = new String(source);
           result = get(lex());
           // If a JSON string contains multiple tokens, it is invalid.
           if (lex() != "$") {