OpenHFT / Chronicle-Values

http://chronicle.software
Other
104 stars 38 forks source link

Something not implemented? #16

Closed bensku closed 7 years ago

bensku commented 7 years ago

Basically, this error... http://hastebin.com/amoporufip.pl

I wonder, what is not implemented yet, exactly and is it going to be implemented soon? Contents of my value class: http://hastebin.com/esexezoday.java

Even knowing what I'd need to change in my value class to get it working would help.

peter-lawrey commented 7 years ago

What is the error you are seeing?

On 22 October 2016 at 19:12, bensku notifications@github.com wrote:

Basically, this error... hastebin.com/esexezoday.java

I wonder, what is not implemented yet, exactly and is it going to be implemented soon? Contents of my value class: http://hastebin.com/esexezoday.java

Even knowing what I'd need to change in my value class to get it working would help.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/OpenHFT/Chronicle-Values/issues/16, or mute the thread https://github.com/notifications/unsubscribe-auth/ABBU8eSUBORyTnPD4zb4k71s5WtGgVWbks5q2lIOgaJpZM4Kd7dq .

leventov commented 7 years ago

@bensku it seems like a bug, or a usability problem at least. But in your case changing to max = 4294967295L might just work, and probably that is what you was intended, if you need "unsigned int". Because the max argument of @Range is included into the range.

bensku commented 7 years ago

It worked, thanks! A bit more useful error message might have helped, though. Closing this...