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