D7096: Only send active window changes to X11 root window if the X11 window changed

Thomas Lübking noreply at phabricator.kde.org
Sun Aug 20 07:20:35 UTC 2017


luebking added a comment.


  David has some point though - m_activeWindow *can* get out of sync (server error, mal... stupid client - and will be temporarily due to the async setup) and must not be used directly to query the active window.
  
  Since there should be only one root per process, maybe rather use a local static than a member (to constrain this cache to the particular function)?

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D7096

To: graesslin, #kwin, #plasma, davidedmundson
Cc: luebking, davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170820/82c30e60/attachment.html>


More information about the Plasma-devel mailing list