D6630: [kstyle] Do not delete the Surface for a QWindow

David Edmundson noreply at phabricator.kde.org
Tue Jul 11 15:54:39 UTC 2017


davidedmundson added a comment.


  Whilst this does make sense based on the current design we are effectively building up dead objects during the lifespan of a window doing show/hide/show.
  
  Someone somewhere should delete it. Assuming everyone follows the same design pattern of following QWindow::hideEvent everyone calling deleteLater() would be safe, but it relies on all parties dealing with surfaces to do that.

REPOSITORY
  R31 Breeze

BRANCH
  master

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

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


More information about the Plasma-devel mailing list