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

[Tejesh R]

On Wed, 19 Mar 2025 07:33:36 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> 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..
> 
>> And why do you want to use local variable ? I don't think local variable would be helpful here.
> 
> I disagree..Using class variable directly might cause synchronization issue which is mitigated by using local var..

I meant w.r.t modifications applied to class and local reference. If both are pointing to same variable you can minimise passing it as parameter in methods selectColumns/Rows. Just a suggestion.

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

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