On Fri, 16 May 2025 21:14:17 GMT, Phil Race <p...@openjdk.org> wrote:
> I think we discussed the performance when it was added (yes we did I just > checked) even though at that time it was only for SystemTray and it was noted > then that DBUS would be a better solution : > https://github.com/openjdk/jdk/pull/17860#discussion_r1490283265 Oh, thanks for the reminder. > but its still not on a "start up" path It's not exactly in there, but it might be pretty close. It's being [called](https://github.com/openjdk/jdk/commit/f75e0ed7c313700044557db6ea09cfd9e85eac70#diff-bc685def01d3c123e569dec4b2dfe03163da3853321e94d2fa81f93e049412c2L1524) now during the static initialization of `MouseEvent`, for example, when the pointer is moved to a frame by a user: java.lang.Exception: Stack trace at java.base/java.lang.Thread.dumpStack(Thread.java:1991) at java.desktop/sun.awt.X11.XToolkit.getNumberOfButtons(XToolkit.java:1526) at java.desktop/java.awt.event.MouseEvent.<clinit>(MouseEvent.java:404) at java.desktop/sun.awt.X11.XWindow.handleXCrossingEvent(XWindow.java:995) at java.desktop/sun.awt.X11.XComponentPeer.handleXCrossingEvent(XComponentPeer.java:74) at java.desktop/sun.awt.X11.XBaseWindow.dispatchEvent(XBaseWindow.java:1155) at java.desktop/sun.awt.X11.XBaseWindow.dispatchToWindow(XBaseWindow.java:1113) at java.desktop/sun.awt.X11.XToolkit.dispatchEvent(XToolkit.java:592) at java.desktop/sun.awt.X11.XToolkit.run(XToolkit.java:705) at java.desktop/sun.awt.X11.XToolkit.run(XToolkit.java:621) at java.base/java.lang.Thread.run(Thread.java:1447) It's still not critical though. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25265#discussion_r2093712590