On Mon, 29 Jan 2024 16:42:15 GMT, Nir Lisker <nlis...@openjdk.org> wrote:

>> Added a utility method to run code on the FX thread if it's not already, and 
>> changed the animation methods to use it.
>
> Nir Lisker has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Update tests

tests/system/src/test/java/test/com/sun/javafx/animation/SynchronizationTest.java
 line 80:

> 78: 
> 79:     private Thread thread;
> 80:     private Throwable throwable;

Since these can be set from any thread, maybe use AtomicReference?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1352#discussion_r1470071779

Reply via email to