Code Refactors

src/main/java/org/json/Cookie.java

@@ -18,7 +18,7 @@ public class Cookie {
     /**
      * Constructs a new Cookie object.
      */
-    public Cookie() {
+    private Cookie() {
     }
 
     /**
@@ -189,21 +189,30 @@ public static String toString(JSONObject jo) throws JSONException {
      * @return The unescaped string.
      */
     public static String unescape(String string) {
+        int i = 0;
         int length = string.length();
         StringBuilder sb = new StringBuilder(length);
-        for (int i = 0; i < length; ++i) {
+
+        while (i < length) {
             char c = string.charAt(i);
             if (c == '+') {
-                c = ' ';
+                sb.append(' ');
+                i++;
             } else if (c == '%' && i + 2 < length) {
                 int d = JSONTokener.dehexchar(string.charAt(i + 1));
                 int e = JSONTokener.dehexchar(string.charAt(i + 2));
+
                 if (d >= 0 && e >= 0) {
-                    c = (char)(d * 16 + e);
-                    i += 2;
+                    sb.append((char)(d * 16 + e));
+                    i += 3;
+                } else {
+                    sb.append(c);
+                    i++;
                 }
+            } else {
+                sb.append(c);
+                i++;
             }
-            sb.append(c);
         }
         return sb.toString();
     }

src/main/java/org/json/CookieList.java

@@ -14,7 +14,7 @@ public class CookieList {
     /**
      * Constructs a new CookieList object.
      */
-    public CookieList() {
+    private CookieList() {
     }
 
     /**

src/main/java/org/json/JSONML.java

@@ -17,9 +17,10 @@ public class JSONML {
     /**
      * Constructs a new JSONML object.
      */
-    public JSONML() {
+    private JSONML() {
     }
 
+
     /**
      * Parse XML values and store them in a JSONArray.
      * @param x       The XMLTokener containing the source string.
@@ -239,9 +240,21 @@ private static Object parse(
                 }
             } else {
                 if (ja != null) {
-                    ja.put(token instanceof String
-                        ? (config.isKeepStrings() ? XML.unescape((String)token) : XML.stringToValue((String)token))
-                        : token);
+                    Object value;
+
+                    if (token instanceof String) {
+                        String strToken = (String) token;
+                        if (config.isKeepStrings()) {
+                            value = XML.unescape(strToken);
+                        } else {
+                            value = XML.stringToValue(strToken);
+                        }
+                    } else {
+                        value = token;
+                    }
+
+                    ja.put(value);
+
                 }
             }
         }