On Mon, 22 Aug 2022 15:43:28 GMT, Jeanette Winzenburg <faste...@openjdk.org> wrote:
>> There is also an unguarded access in CellBehaviorBase.selectRows(int,int) : >> 312 and CellBehaviorBase.sompleSelect(MouseButton,int,boolean) : 274, which >> are being fixed by #876 >> >> I'd suggest to integrate #876 first, followed by this PR, in order to avoid >> merge conflicts. > >> There is also an unguarded access in CellBehaviorBase.selectRows(int,int) : >> 312 and CellBehaviorBase.sompleSelect(MouseButton,int,boolean) : 274, which >> are being fixed by #876 >> >> I'd suggest to integrate #876 first, followed by this PR, in order to avoid >> merge conflicts. > > @andy-goryachev-oracle hmm .. are those methods reached if selectionModel == > null? On skimming across its code, it looks like all callers back out (that > is not call it) without selectionModel. Might be wrong, though - hard to tell > without documented preconditions, and without tests 😉 @kleopatra : simpleSelect and selectRows are being called from CellBehaviorBase.mousePressed and mouseReleased, so probably yes. in any way, they are updated in #876 ; fixing them here would likely create a merge conflict. ------------- PR: https://git.openjdk.org/jfx/pull/711