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

Reply via email to