On Fri, 3 Sep 2021 18:19:45 GMT, Michael Strauß <mstra...@openjdk.org> wrote:

> This PR fixes the exception thrown by the sample code in 
> [8273324](https://bugs.openjdk.java.net/browse/JDK-8273324), while retaining 
> the incorrect behavior in the scenario described.

Marked as reviewed by jpereda (Committer).

Looks good to me too. 

It solves the exception, and also solves the incorrect null element that was 
logged:
`Change: { [null] removed at 1 }, selected items: []` now reports correctly as 
`Change: { [Person{a b}] removed at 0 },  selected items: []` ).

(However, the deselected cell is still showing as selected, but that is for 
[JDK-8273336](https://bugs.openjdk.java.net/browse/JDK-8273336).)

-------------

PR: https://git.openjdk.java.net/jfx/pull/617

Reply via email to