Integrated: 8279514: NPE on clearing value of IntegerSpinnerValueFactory
Ajit Ghaisas
aghaisas at openjdk.org
Fri Sep 2 03:18:02 UTC 2022
On Wed, 10 Aug 2022 06:56:50 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:
> 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.
This pull request has now been integrated.
Changeset: d1a7ebdc
Author: Ajit Ghaisas <aghaisas at openjdk.org>
URL: https://git.openjdk.org/jfx/commit/d1a7ebdc3374066f1c1f2739551185cbb34bdf96
Stats: 46 lines in 2 files changed: 46 ins; 0 del; 0 mod
8279514: NPE on clearing value of IntegerSpinnerValueFactory
Reviewed-by: mhanl, kcr
-------------
PR: https://git.openjdk.org/jfx/pull/865
More information about the openjfx-dev
mailing list