On Tue, 21 Nov 2023 14:38:39 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Tejesh R has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review fix
>
> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTableUI.java line 
> 2031:
> 
>> 2029:             damagedArea.x = 
>> SwingUtilities2.getXPosInRightToLeft(table, cMin);
>> 2030:         } else {
>> 2031:             damagedArea.x = 
>> SwingUtilities2.getXPosInRightToLeft(table, cMax);
> 
> The method name `getXPosInRightToLeft` is confusing when you call it for the 
> *left-to-right case*…

Yeah, any suggestions?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16374#discussion_r1401577410

Reply via email to