D5723: Cleanup wayland resources used in kwindowsystem before they're deleted by QPA

Martin Gräßlin noreply at phabricator.kde.org
Sat May 6 04:56:34 UTC 2017


graesslin accepted this revision.
graesslin added a comment.
This revision is now accepted and ready to land.


  This is genius, using release! I spent hours on thinking how to properly clean that up and always came to the conclusion it's not possible as we cannot properly clean up because we don't get informed when the connection is destroyed.

INLINE COMMENTS

> waylandintegration.cpp:79
> +        if (m_waylandContrastManager) {
> +           m_waylandContrastManager->release();
> +        }

nitpick: indentation

REPOSITORY
  R130 Frameworks integration plugin using KWayland

BRANCH
  master

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

To: davidedmundson, #plasma, graesslin
Cc: graesslin, cfeck, plasma-devel, spstarr, progwolff, Zren, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170506/873c60cd/attachment.html>


More information about the Plasma-devel mailing list