Made getters final; added deepCopy() to PathValueOperation.getValue()

Author: GavinF17

src/main/java/com/github/fge/jsonpatch/DualPathOperation.java

@@ -72,7 +72,7 @@ public final void serializeWithType(final JsonGenerator jgen,
         serialize(jgen, provider);
     }
 
-    public JsonPointer getFrom() {
+    public final JsonPointer getFrom() {
         return from;
     }
 

src/main/java/com/github/fge/jsonpatch/JsonPatch.java

@@ -151,7 +151,7 @@ public JsonNode apply(final JsonNode node)
         return ret;
     }
 
-    public List<JsonPatchOperation> getOperations() {
+    public final List<JsonPatchOperation> getOperations() {
         return operations;
     }
 

src/main/java/com/github/fge/jsonpatch/JsonPatchOperation.java

@@ -94,11 +94,11 @@ protected JsonPatchOperation(final String op, final JsonPointer path)
     public abstract JsonNode apply(final JsonNode node)
         throws JsonPatchException;
 
-    public String getOp() {
+    public final String getOp() {
         return op;
     }
 
-    public JsonPointer getPath() {
+    public final JsonPointer getPath() {
         return path;
     }
 

src/main/java/com/github/fge/jsonpatch/PathValueOperation.java

@@ -73,8 +73,8 @@ public final void serializeWithType(final JsonGenerator jgen,
         serialize(jgen, provider);
     }
 
-    public JsonNode getValue() {
-        return value;
+    public final JsonNode getValue() {
+        return value.deepCopy();
     }
 
     @Override