On Wed, 28 Apr 2021 10:50:29 GMT, Jeanette Winzenburg <faste...@openjdk.org> 
wrote:

> There is one more issue mentioned (2nd comment by OP): toggling selection 
> throws an NPE - that's still the case, both with and without this PR.

*hach .. that's fixed by the most recent change in MultipleSelectionModelBase 
(wasn't aware you did change sooo much ;)

The boundaries of this fix are getting a bit too blurry (as is the other bug 
report [JDK-8189228](https://bugs.openjdk.java.net/browse/JDK-8189228) ;) for 
my taste.  

Options either:

- keep this focused on the IOOB error
- find/open an new issue about the incorrect clearSelection impl in 
MultipleSelectionModelBase

or:
- explicityl widen the scope of this to include the clearSelection bug

My preference would be the first.

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

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

Reply via email to