On Wed, 10 Aug 2022 06:56:50 GMT, Ajit Ghaisas <[email protected]> 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 <[email protected]>
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

Reply via email to