RFR: 8279514: NPE on clearing value of IntegerSpinnerValueFactory
Ajit Ghaisas
aghaisas at openjdk.org
Wed Aug 10 07:03:10 UTC 2022
A null check has been added to below SpinnerValueFactory classes in valueProperty() ChangeListeners-
- `IntegerSpinnerValueFactory`
- `LocalDateSpinnerValueFactory`
- `LocalTimeSpinnerValueFactory`
- `ListSpinnerValueFactory`
Added 5 tests that detect NPE for above 4 Spinner factories.
Only `DoubleSpinnerValueFactory` already had a valid null check - so the test for it passes before and after.
-------------
Commit messages:
- Add null checks to SpinnerValueFactories
Changes: https://git.openjdk.org/jfx/pull/865/files
Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=865&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8279514
Stats: 46 lines in 2 files changed: 46 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jfx/pull/865.diff
Fetch: git fetch https://git.openjdk.org/jfx pull/865/head:pull/865
PR: https://git.openjdk.org/jfx/pull/865
More information about the openjfx-dev
mailing list