On Tue, 10 Jun 2025 11:32:58 GMT, Anass Baya <ab...@openjdk.org> wrote:
>> check how the _SetFocusableWindow is implemented below. > > Hello @mrserb, > >> I think the only thing you should do is to replace the usage of >> JNI_GET_PDATA by the JNI_CHECK_PEER_GOTO and add a label "ret" at the end so >> default value of "result" will be returned. > > Yes, In the latest changes, I followed that approach to ensure that the peer > is not yet destroyed in _GetScreenImOn() as explain in my latest comment. So > if you agree I will remove the lock. Yes, the new lock is not needed. You can verify this by adding delays/Sleep calls into the methods and confirming that they wait for each other. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25619#discussion_r2138797641
