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

Reply via email to