On Wed, 2 Sep 2026 22:21:28 GMT, Andy Goryachev <[email protected]> wrote:

>> Kevin Rushforth has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   rm extra newlines
>
> tests/system/src/test/java/test/com/sun/glass/ui/WinTimerTest.java line 167:
> 
>> 165:         }
>> 166:         assertFalse(stopperThread.isAlive(), "Stopper thread is still 
>> running");
>> 167:         assertNull(throwable.get());
> 
> should probably verify that the timer is not running at this point

Good idea. Added. I also added an assert that the timer is running after it is 
first started.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2290#discussion_r3919853114

Reply via email to