astyanax icon indicating copy to clipboard operation
astyanax copied to clipboard

fromByteBuffer() not working properly in Int32Serializer/IntegerSerializer

Open blop opened this issue 10 years ago • 2 comments

Hello,

It seems to me fromByteBuffer() should work with bytebuffers with more bytes remaining too, as it is only reading the next 4 bytes.

Also, the instance field in Int32Serializer was not initialized correctly.

Thanks.

Olivier

blop avatar Aug 04 '14 14:08 blop

This makes fromByteBuffer() of Int32Serializer/IntegerSerializer more coherent with BooleanSerializer behaviour.

blop avatar Aug 04 '14 14:08 blop

Actually, I added a small fix for LongSerializer too.

blop avatar Aug 04 '14 14:08 blop