On Wed, 5 Aug 2026 17:49:27 GMT, Andy Goryachev <[email protected]> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Use state check
>
> test/jdk/javax/swing/JTable/TestAutoResizeLastColumn.java line 44:
> 
>> 42:         table.setAutoResizeMode(JTable.AUTO_RESIZE_LAST_COLUMN);
>> 43: 
>> 44:         table.getTableHeader().setResizingColumn(null);
> 
> is this line necessary for the test?  
> 
> in my test app (modeled after this test) it works fine if I leave this line 
> uncommented.  I am running 25.0.2+10-69

it was needed to isolate one layout path to force “no active header resize” 
path but since the old fix is removed, then after
table.setAutoResizeMode(JTable.AUTO_RESIZE_LAST_COLUMN);
header’s resizingColumn should naturally be null..
I have modified to check for the state instead so it should catch both issues
- AUTO_RESIZE_LAST_COLUMN should resize only the last column during normal 
table/window resize.
- setAutoResizeMode(AUTO_RESIZE_LAST_COLUMN) must not fake an active header 
resize
and it fails in jdk25 interim build

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31704#discussion_r3725674192

Reply via email to