D15295: [konsole]: proper fix for a crash-on-exit

[René J.V. Bertin]

rjvbb added a comment.


  I think these do different things; containerEmpty is called when a container (Konsole window, presumably) closed its last tab but doesn't unregister the container. I'd guess that containerEmpty will be called first, and then containerDestroyed when the container is being disposed off (because it's empty). 
  There may not be an explicit current need to handle these separately, but I don't see why one should NOT do that.
  
  Note that handling them separately will be necessary if container (Konsole window) is deleted via deleteLater() or otherwise not immediately after closing the last tab.

REPOSITORY
  R319 Konsole

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

To: rjvbb, #konsole, gateau, hindenburg
Cc: tcanabrava, konsole-devel, #kde_applications, herrold, ngraham, maximilianocuria, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20180905/36bffde4/attachment-0001.html>