refactor-strict-mode: remove unneeded param for parseTokener()

stleary · May 27, 2024 · 4335997 · 4335997
1 parent 2896cbd
commit 4335997
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/src/main/java/org/json/JSONArray.java b/src/main/java/org/json/JSONArray.java
@@ -110,16 +110,14 @@ public JSONArray(JSONTokener jsonTokener, JSONParserConfiguration jsonParserConf
             throw this.jsonTokener.syntaxError("A JSONArray text must start with '['");
         }
 
-        parseTokener(jsonParserConfiguration); // runs recursively
+        parseTokener(); // runs recursively
 
     }
 
     /**
      * This method utilizes a JSONTokener initialized with a string to parse a JSONArray
-     * @param jsonParserConfiguration
      */
-    private void parseTokener(JSONParserConfiguration jsonParserConfiguration) {
-        boolean strictMode = jsonParserConfiguration.isStrictMode();
+    private void parseTokener() {
 
         char cursor = this.jsonTokener.nextClean();
 
@@ -132,7 +130,7 @@ private void parseTokener(JSONParserConfiguration jsonParserConfiguration) {
 
                 throwErrorIfEoF();
 
-                if(strictMode && cursor == ']'){
+                if(this.jsonParserConfiguration.isStrictMode() && cursor == ']'){
                     throw this.jsonTokener.syntaxError(getInvalidCharErrorMsg(cursor));
                 }
 
@@ -142,7 +140,7 @@ private void parseTokener(JSONParserConfiguration jsonParserConfiguration) {
 
                 this.jsonTokener.back();
 
-                parseTokener(jsonParserConfiguration);
+                parseTokener();
                 break;
             case ']':
                 break;
@@ -151,7 +149,7 @@ private void parseTokener(JSONParserConfiguration jsonParserConfiguration) {
                 boolean currentCharIsQuote = this.jsonTokener.getPrevious() == '"';
 
                 this.myArrayList.add(this.jsonTokener.nextValue(jsonParserConfiguration));
-                parseTokener(jsonParserConfiguration);
+                parseTokener();
         }
     }