RFR: 8354095: Open some JTable bugs 5 [v3]

Mon Apr 21 20:51:48 UTC 2025

On Mon, 21 Apr 2025 20:25:51 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Alisen Chung has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   manual jtreg tag, make jtable field local
>
> test/jdk/javax/swing/JTable/bug4118307.java line 50:
> 
>> 48:     static Point tableLoc;
>> 49:     static Point p;
>> 50:     private static boolean flag = false;
> 
> should we mark it as volatile or is it somehow synced?

I agree. 'flag' can be made volatile and directly used in the test. This eliminates the need for setFail(), isFail() synchronized functions below.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24734#discussion_r2052968371