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