On Mon, 29 Jan 2024 19:05:46 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> 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?

Alright, but because of the latch I don't think it will ever matter in practice.

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

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

Reply via email to