Change access modifier to protected in case DefaultComparator gets inherited

Tamas Balog · Tamas Balog · commit ecd41b9c31b1 · 2017-02-20T07:01:52.000+01:00
diff --git a/src/main/java/org/skyscreamer/jsonassert/comparator/DefaultComparator.java b/src/main/java/org/skyscreamer/jsonassert/comparator/DefaultComparator.java
@@ -1,14 +1,14 @@
 package org.skyscreamer.jsonassert.comparator;
 
-import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allJSONObjects;
-import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allSimpleValues;
-
 import org.json.JSONArray;
 import org.json.JSONException;
 import org.json.JSONObject;
 import org.skyscreamer.jsonassert.JSONCompareMode;
 import org.skyscreamer.jsonassert.JSONCompareResult;
 
+import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allJSONObjects;
+import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allSimpleValues;
+
 /**
  * This class is the default json comparator implementation. <p/>
  * Comparison is performed according to {@link JSONCompareMode} that is passed as constructor's argument.
@@ -75,11 +75,11 @@ public void compareJSONArray(String prefix, JSONArray expected, JSONArray actual
         }
     }
 
-    private boolean areNumbers(Object expectedValue, Object actualValue) {
+    protected boolean areNumbers(Object expectedValue, Object actualValue) {
         return expectedValue instanceof Number && actualValue instanceof Number;
     }
 
-    private boolean areSameDoubles(Object expectedValue, Object actualValue) {
+    protected boolean areSameDoubles(Object expectedValue, Object actualValue) {
         return ((Number) expectedValue).doubleValue() != ((Number) actualValue).doubleValue();
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -1,14 +1,14 @@`
`1`	`1`	`package org.skyscreamer.jsonassert.comparator;`
`2`	`2`
`3`		`-import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allJSONObjects;`
`4`		`-import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allSimpleValues;`
`5`		`-`
`6`	`3`	`import org.json.JSONArray;`
`7`	`4`	`import org.json.JSONException;`
`8`	`5`	`import org.json.JSONObject;`
`9`	`6`	`import org.skyscreamer.jsonassert.JSONCompareMode;`
`10`	`7`	`import org.skyscreamer.jsonassert.JSONCompareResult;`
`11`	`8`
	`9`	`+import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allJSONObjects;`
	`10`	`+import static org.skyscreamer.jsonassert.comparator.JSONCompareUtil.allSimpleValues;`
	`11`	`+`
`12`	`12`	`/**`
`13`	`13`	`* This class is the default json comparator implementation. <p/>`
`14`	`14`	`* Comparison is performed according to {@link JSONCompareMode} that is passed as constructor's argument.`
`@@ -75,11 +75,11 @@ public void compareJSONArray(String prefix, JSONArray expected, JSONArray actual`
`75`	`75`	`}`
`76`	`76`	`}`
`77`	`77`
`78`		`- private boolean areNumbers(Object expectedValue, Object actualValue) {`
	`78`	`+ protected boolean areNumbers(Object expectedValue, Object actualValue) {`
`79`	`79`	`return expectedValue instanceof Number && actualValue instanceof Number;`
`80`	`80`	`}`
`81`	`81`
`82`		`- private boolean areSameDoubles(Object expectedValue, Object actualValue) {`
	`82`	`+ protected boolean areSameDoubles(Object expectedValue, Object actualValue) {`
`83`	`83`	`return ((Number) expectedValue).doubleValue() != ((Number) actualValue).doubleValue();`
`84`	`84`	`}`
`85`	`85`	`}`