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

Reply via email to