On Mon, 21 Apr 2025 20:25:51 GMT, Sergey Bylokhov <s...@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
