On Tue, 31 Mar 2026 17:03:19 GMT, Alexander Zvegintsev <[email protected]> 
wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8380578
>
> src/java.desktop/unix/classes/sun/awt/X11/XToolkit.java line 632:
> 
>> 630:                 // for compatibility reasons. Probably some time later 
>> we'll remove
>> 631:                 // the check for AppContext.isDisposed() and will 
>> unconditionally
>> 632:                 // break the loop here.
> 
> It looks like this if block is no longer skipping anything and can be removed

I kept this in case there are reasons other than the disposal of an AppContext 
which might interrupt the thread. I didn't want to risk an incompatible change.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30362#discussion_r3017655436

Reply via email to