On Mon, 14 Nov 2022 14:38:38 GMT, Alexander Zvegintsev <[email protected]> 
wrote:

>> This test currently passing as it is on all platforms, but it was 
>> problemlisted with ["Problem List some tests that leave windows open on the 
>> desktop" 
>> reason](https://github.com/openjdk/jdk/commit/fd8df3adf13f5df8f5f95482b8e0a70bc519f39f)
>> 
>> So this test is cleaned up and stabilized.
>> 
>> CI testing looks good.
>
> Alexander Zvegintsev has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Addressing review comments

So - apart from the last change to use volatile - you found nothing 
functionally wrong with the test ?
There's so much clean up I can't be sure I'm not missing some functional change 
you made.
If there was nothing wrong with it, why was it leaving windows open ?

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

PR: https://git.openjdk.org/jdk/pull/11128

Reply via email to