RFR: 8193286: IntegerSpinnerFactory does not wrap value correctly
Ajit Ghaisas
aghaisas at openjdk.java.net
Wed Apr 22 12:02:30 UTC 2020
On Wed, 22 Apr 2020 09:06:09 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:
>> I checked and there is a case that (mostly) works today that will break with your proposed fix. I left a couple inline
>> comments.
>> I wonder if it is better to wait and fix it completely in
>> [JDK-8242553](https://bugs.openjdk.java.net/browse/JDK-8242553).
>
>>
>> I wonder if it is better to wait and fix it completely in
>> [JDK-8242553](https://bugs.openjdk.java.net/browse/JDK-8242553).
>
> good idea - do it correctly once and for all :)
Thanks @kevinrushforth for taking a detailed look at this.
I wanted to fix this and then fix the buggy behavior change in JDK-8242553 separately.
As my proposed Spinner.wrapValue() does not work well in some cases and it's going to get modified anyway - I guess,
you and @kleopatra are right in suggesting to fix it entirely in JDK-8242553. I will close this PR.
-------------
PR: https://git.openjdk.java.net/jfx/pull/174
More information about the openjfx-dev
mailing list