On Mon, 21 Apr 2025 23:02:13 GMT, Alisen Chung <ach...@openjdk.org> wrote:

>> Opening some JComboBox bugs
>
> Alisen Chung has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   add volatile

test/jdk/javax/swing/JComboBox/bug4436376.java line 44:

> 42:     static JFrame frame;
> 43:     static volatile Point p;
> 44:     static volatile JComboBox combo;

Making Swing object volatile does not mean that calling its methods not from 
EDT is Ok. As a solution you can perform the entire check block together with 
throwing an exception inside the invokeAndWait block. Same with the second test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24767#discussion_r2054351732

Reply via email to