RFR: 8138842: TableViewSelectionModel.selectIndices does not select index 0 [v2]

Ajit Ghaisas aghaisas at openjdk.org
Mon Feb 6 06:30:57 UTC 2023


On Wed, 1 Feb 2023 06:37:21 GMT, Karthik P K <kpk at openjdk.org> wrote:

>> In `selectIndices` method, zero length array is not considered while ignoring row number given as parameter.
>> 
>> Updated the code to consider both null and zero length array in the condition before ignoring the row value given as parameter.
>> 
>> Added unit test to validate the fix
>
> Karthik P K has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix first index selection issue in TreeTableView

Looks good to me.

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

Marked as reviewed by aghaisas (Reviewer).

PR: https://git.openjdk.org/jfx/pull/1018


More information about the openjfx-dev mailing list