msgpack#100 Fixes readLen computation in unpackString

xerial · xerial · commit 2657f070b741 · 2014-06-02T12:31:36.000+09:00
diff --git a/msgpack-core/src/main/java/org/msgpack/core/MessageUnpacker.java b/msgpack-core/src/main/java/org/msgpack/core/MessageUnpacker.java
@@ -825,8 +825,8 @@ public String unpackString() throws IOException {
                             cr = decoder.flush(decodeBuffer);
 
                         if(cr.isOverflow()) {
-                            // There is insufficient space in the output CharBuffer.
-                            readLen = bb.position() - bb.remaining();
+                            // The output CharBuffer has insufficient space
+                            readLen = bb.limit() - bb.remaining();
                         }
 
                         if(cr.isError()) {

Original file line number	Diff line number	Diff line change
`@@ -825,8 +825,8 @@ public String unpackString() throws IOException {`
`825`	`825`	`cr = decoder.flush(decodeBuffer);`
`826`	`826`
`827`	`827`	`if(cr.isOverflow()) {`
`828`		`- // There is insufficient space in the output CharBuffer.`
`829`		`- readLen = bb.position() - bb.remaining();`
	`828`	`+ // The output CharBuffer has insufficient space`
	`829`	`+ readLen = bb.limit() - bb.remaining();`
`830`	`830`	`}`
`831`	`831`
`832`	`832`	`if(cr.isError()) {`