RFR: 4466930: JTable.selectAll boundary handling [v7]

Tejesh R tr at openjdk.org
Thu Mar 20 05:30:13 UTC 2025


On Wed, 19 Mar 2025 13:49:29 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> JTable.selectAll doesn't do anything if there are no rows or no columns. 
>> But it should still select all columns if there are no rows and the other way round. 
>> It is seen that isColumnSelected() will return false for all columns after calling selectAll() if there happened to be no rows.
>> 
>> Fix is made to select all columns even if there are no rows and similarly for rows if there are no columns.
>
> 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);

            }`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24025#discussion_r2004835032


More information about the client-libs-dev mailing list