RFR: 4466930: JTable.selectAll boundary handling [v7]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Thu Mar 20 05:35:13 UTC 2025
On Thu, 20 Mar 2025 05:27:59 GMT, Tejesh R <tr at openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add check for rowselection and columnselection
>
> src/java.desktop/share/classes/javax/swing/JTable.java line 2198:
>
>> 2196: if (getColumnSelectionAllowed()) {
>> 2197: selectColumns(columnCount);
>> 2198: }
>
> Can we simplify the code from line 2188 - 2198, since the intension is to selectRows/Columns if its available.
> `
> if (rowCount > 0 ) {
>
> if (getRowSelectionAllowed()) {
>
> selectRows(rowCount);
>
> }
>
> } else if (columnCount > 0 ) {
>
> if (getColumnSelectionAllowed()) {
>
> selectColumns(columnCount);
>
> }`
No that might change the existing behavior and JCK expectation for row>0 and column>0 if we introduce the check..
SO, I will like to keep it as it is..
You can raise a followon bug if you would like it to be worked on later..
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24025#discussion_r2004838801
More information about the client-libs-dev
mailing list