Code cleanup according to merge request comments.

Author: pgaspar7impresa

jmespath-jakarta-jsonp/src/main/java/io/burt/jmespath/jakarta/jsonp/JsonpRuntime.java

@@ -74,11 +74,11 @@ public List<JsonValue> toList(JsonValue value) {
     if (valueType == ARRAY) {
       return new JsonArrayListWrapper((JsonArray) value);
     } else if (valueType == OBJECT) {
-      JsonObject obj = (JsonObject)value;
+      JsonObject obj = (JsonObject) value;
       if (!obj.isEmpty()) {
         List<JsonValue> elements = new ArrayList<>(obj.size());
         for (JsonValue v : obj.values()) {
-          elements.add(useJsonNull(v));
+          elements.add(nodeOrNullNode(v));
         }
         return elements;
       }
@@ -89,15 +89,15 @@ public List<JsonValue> toList(JsonValue value) {
   @Override
   public String toString(JsonValue str) {
     if (str.getValueType() == STRING) {
-      return ((JsonString)str).getString();
+      return ((JsonString) str).getString();
     } else {
       return str.toString();
     }
   }
 
   @Override
   public Number toNumber(JsonValue n) {
-    return (n.getValueType() == NUMBER) ? ((JsonNumber)n).numberValue() : null;
+    return (n.getValueType() == NUMBER) ? ((JsonNumber) n).numberValue() : null;
   }
 
   @Override
@@ -110,11 +110,11 @@ public boolean isTruthy(JsonValue value) {
       case TRUE:
         return true;
       case ARRAY:
-        return ((JsonArray)value).size() > 0;
+        return ((JsonArray) value).size() > 0;
       case OBJECT:
-        return ((JsonObject)value).size() > 0;
+        return ((JsonObject) value).size() > 0;
       case STRING:
-        return ((JsonString)value).getString().length() > 0;
+        return ((JsonString) value).getString().length() > 0;
       default:
         throw new IllegalStateException(String.format("Unknown node type encountered: %s", value.getValueType()));
     }
@@ -144,7 +144,7 @@ public JmesPathType typeOf(JsonValue value) {
   @Override
   public JsonValue getProperty(JsonValue value, JsonValue name) {
     if (value.getValueType() == OBJECT) {
-      return nodeOrNullNode(((JsonObject)value).get(textOnStringOrNull(name)));
+      return nodeOrNullNode(((JsonObject) value).get(toString(name)));
     } else {
       return JsonValue.NULL;
     }
@@ -153,7 +153,7 @@ public JsonValue getProperty(JsonValue value, JsonValue name) {
   @Override
   public Collection<JsonValue> getPropertyNames(JsonValue value) {
     if (value.getValueType() == OBJECT) {
-      Set<String> nameSet = ((JsonObject)value).keySet();
+      Set<String> nameSet = ((JsonObject) value).keySet();
       List<JsonValue> names = new ArrayList<>(nameSet.size());
       for(String n : nameSet) {
         names.add(createString(n));
@@ -173,14 +173,14 @@ public JsonValue createNull() {
   public JsonValue createArray(Collection<JsonValue> elements) {
     JsonArrayBuilder builder = Json.createArrayBuilder();
     for(JsonValue element : elements) {
-      builder.add(useJsonNull(element));
+      builder.add(nodeOrNullNode(element));
     }
     return builder.build();
   }
 
   @Override
   public JsonValue createString(String str) {
-    return useJsonNull(Json.createValue(str));
+    return nodeOrNullNode(Json.createValue(str));
   }
 
   @Override
@@ -192,9 +192,9 @@ public JsonValue createBoolean(boolean b) {
   public JsonValue createObject(Map<JsonValue, JsonValue> obj) {
     JsonObjectBuilder builder = Json.createObjectBuilder();
     for (Map.Entry<JsonValue, JsonValue> entry : obj.entrySet()) {
-      String key = textOnStringOrNull(entry.getKey());
+      String key = toString(entry.getKey());
       if (key != null) {
-        builder.add(key, useJsonNull(entry.getValue()));
+        builder.add(key, nodeOrNullNode(entry.getValue()));
       }
     }
     return builder.build();
@@ -210,21 +210,11 @@ public JsonValue createNumber(long n) {
     return Json.createValue(n);
   }
 
-  private JsonValue useJsonNull(JsonValue value) { return (value == null) ? JsonValue.NULL : value; }
-
   private JsonValue nodeOrNullNode(JsonValue node) {
     if (node == null) {
       return JsonValue.NULL;
     } else {
       return node;
     }
   }
-
-  private String textOnStringOrNull(JsonValue value) {
-    if (value.getValueType() == STRING) {
-      return ((JsonString)value).getString();
-    } else {
-      return null;
-    }
-  }
 }