Update to include error location when creating JSONObject from string/text

migueltt · migueltt · commit 7fed02308060 · 2017-08-09T21:52:36.000-04:00
diff --git a/JSONObject.java b/JSONObject.java
@@ -231,7 +231,21 @@ public JSONObject(JSONTokener x) throws JSONException {
             if (c != ':') {
                 throw x.syntaxError("Expected a ':' after a key");
             }
-            this.putOnce(key, x.nextValue());
+            
+            // Replace: this.putOnce(key, x.nextValue());
+            // Use syntaxError(..) to include error location
+            
+            if (key != null) {
+                // Check if key exists
+            	if (this.opt(key) != null) {
+                    throw x.syntaxError("Duplicate key \"" + key + "\"");
+            	}
+                // Only add value if non-null
+                Object value = x.nextValue();
+                if (value!=null) {
+                	this.put(key, value);
+                }
+            }
 
             // Pairs are separated by ','.
 
