isImmutable is false for Array/MapCursor

xerial · xerial · commit 84ac45926d2a · 2014-11-28T18:32:43.000+09:00
diff --git a/msgpack-core/src/main/java/org/msgpack/value/impl/ArrayCursorImpl.java b/msgpack-core/src/main/java/org/msgpack/value/impl/ArrayCursorImpl.java
@@ -118,5 +118,10 @@ public ArrayValue toImmutable() {
             arr[i++] = v.toImmutable();
         }
         return ValueFactory.newArray(arr);
+
     }
+
+    @Override
+    public boolean isImmutable() { return false; }
+
 }
diff --git a/msgpack-core/src/main/java/org/msgpack/value/impl/MapCursorImpl.java b/msgpack-core/src/main/java/org/msgpack/value/impl/MapCursorImpl.java
@@ -125,5 +125,7 @@ public MapValue toImmutable() {
         return ValueFactory.newMap(keyValueArray);
     }
 
+    @Override
+    public boolean isImmutable() { return false; }
 
 }

Original file line number	Diff line number	Diff line change
`@@ -118,5 +118,10 @@ public ArrayValue toImmutable() {`
`118`	`118`	`arr[i++] = v.toImmutable();`
`119`	`119`	`}`
`120`	`120`	`return ValueFactory.newArray(arr);`
	`121`	`+`
`121`	`122`	`}`
	`123`	`+`
	`124`	`+ @Override`
	`125`	`+ public boolean isImmutable() { return false; }`
	`126`	`+`
`122`	`127`	`}`
Original file line number	Diff line number	Diff line change
`@@ -125,5 +125,7 @@ public MapValue toImmutable() {`
`125`	`125`	`return ValueFactory.newMap(keyValueArray);`
`126`	`126`	`}`
`127`	`127`
	`128`	`+ @Override`
	`129`	`+ public boolean isImmutable() { return false; }`
`128`	`130`
`129`	`131`	`}`