Struct.NumberField.getString() downcasts to int about jnr-ffi HOT 4 CLOSED

It does appear that this logic does not support 64-bit values at all. Changing the toString is only one small part of it; note that NumberField.longValue also calls intValue and upcasts it to a long. It's unclear to me if we can just switch all of these classes to actually treat the contained value as a long.

Thoughts?

from jnr-ffi.

AFAICT, NumberField.longValue is always overridden in the places that matter, e.g. IntegerAlias, Signed64, Unsigned64, SignedLong, UnsignedLong. Perhaps you could make all those methods abstract to avoid confusion?

from jnr-ffi.

And calling longValue() will still be wrong if the number is a float. Looks like it's more correct to make the method abstract.

from jnr-ffi.

The shortest fix seems to be to copy over the toString() method of Signed64 / Unsigned64 / SignedLong / UnsignedLong into IntegerAlias and Unsigned32. The delicate series of overrides of longValue() and toString() in derived classes seems to handle every other case.

Alternately we can make longValue() and toString() abstract so that the code is easier to read but lengthier.

from jnr-ffi.