D6571: Delete all Wayland connections by plugins prior to own Wayland connection

[Martin Flöser]

graesslin added inline comments.

INLINE COMMENTS

> davidedmundson wrote in wayland_server.cpp:99
> Alternative.
> 
> Instead of deleting qobjects
> 
> emit c->connectionDied();
> 
> Clients then call destroy() on whatever objects.
> They can then delete the QObjects whenever they were going to.

Yeah that should work as well. Though of course the documentation of ConnectionThread says it's not emitted if created from application, but we can fix that ;-)

It might mean we leak a few resources, but as we are shutting down anyway...

REPOSITORY
  R108 KWin

BRANCH
  connection-cleanup

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

To: graesslin, #kwin, #plasma, mart
Cc: 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/20170715/9ba3ab3c/attachment.html>