On Sun, 22 Mar 2026 22:05:45 GMT, Phil Race <[email protected]> wrote:

> This is to address removing AppContext from various places in the AWT 
> implementation.
> 
> This most substantial is in various XAWT / Toolkit related classes to remove 
> AppContext lookup in event dispatching and related code
> 
> - An unused field of a native XAWT structure references the appContext field 
> of a Component and so it is deleted
> 
> - The same exists in windows, and although nothing to do with XAWT it is 
> cleaned up here
> 
> - one obsolete comment (no code change) is removed from a PerformanceLogger 
> class
> 
> No tests fail.

src/java.desktop/unix/classes/sun/awt/X11/XToolkit.java line 630:

> 628:                 // If the thread is interrupted from another place, 
> let's skip it
> 629:                 // for compatibility reasons. Probably some time later 
> we'll remove
> 630:                 // break the loop here.

The comment still mention the "break" which is now deleted?

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

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

Reply via email to