RFR: 4466930: JTable.selectAll boundary handling [v7]
Tejesh R
tr at openjdk.org
Thu Mar 20 06:29:09 UTC 2025
On Thu, 20 Mar 2025 05:32:22 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> 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..
Sure, raising a follow up bug makes sense.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24025#discussion_r2004901773
More information about the client-libs-dev
mailing list