On Tue, 10 Jun 2025 23:51:09 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:

>> Anass Baya has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Remove global lock
>
> src/java.desktop/windows/native/libawt/windows/awt_Window.cpp line 2507:
> 
>> 2505:     // before our java peer - if we're dispose()d, for instance.
>> 2506:     // Return the default screen.
>> 2507:     JNI_CHECK_PEER_GOTO(self, ret);
> 
> I suggest reordering it slightly - this pattern is commonly used in most 
> cases where JNI_CHECK_PEER_GOTO is used:
> 
>     result...;
>     AwtWindow *window = NULL;
> 
>     PDATA pData;
>     JNI_CHECK_PEER_GOTO(self, ret);
>     window = (AwtWindow *)pData;

I still suggest this order.

Also, please add the new bug ID to the GetGraphicsStressTest.
btw I'm still looking into the new test - it always passes for me. Can we tweak 
it to reproduce the bug more reliably?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25619#discussion_r2139042941

Reply via email to