fix issue 159

[xuwei-k]

ref #159

[xerial]

Hi,

I think reseting the cursor position of ByteBuffer is not common in Java library. For example, ByteBuffer.put(ByteBuffer src) moves the cursor position of the input source after writing the data: http://docs.oracle.com/javase/7/docs/api/java/nio/ByteBuffer.html#put(java.nio.ByteBuffer)

Could you explain a little bit more why resetting the cursor position is necessary?

[xuwei-k]

I just simply want to fix #159. RawStringValueImpl#equals depends on position of ByteBuffer

• https://github.com/msgpack/msgpack-java/blob/9cb9bfae6210a5ecbd/msgpack-core/src/main/java/org/msgpack/value/impl/RawStringValueImpl.java#L57
• xuwei-k@6cbc67494

Should we fix RawStringValueImpl#equals instead of MessagePacker#writePayload ?

[xerial]

Yes. Moving the cursor position of ByteBuffer is necessary to let the user know how many bytes are read from ByteBuffer.