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

Prasanta Sadhukhan psadhukhan at openjdk.org
Wed Mar 19 07:36:09 UTC 2025


On Wed, 19 Mar 2025 07:30:17 GMT, Tejesh R <tr at openjdk.org> wrote:

>> Yes, good catch..we need to use local variable as done in old code and not modify the class variables..Modified to store the instance variable in local and use that..
>
> I didn't mean that, I meant the other way. This would still modify the class variable right? And why do you want to use local variable ? Since you are getting selectionIndex, I don't think local variable would be helpful here.

THis is the way it was done previously where also the action was done on local var..please see previous code....the selectionIndex was obtained from selectionModel class var..

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

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


More information about the client-libs-dev mailing list