<table><tr><td style="">rjvbb added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15295">View Revision</a></tr></table><br /><div><div><p>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). <br />
There may not be an explicit current need to handle these separately, but I don't see why one should NOT do that.</p>

<p>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.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15295">https://phabricator.kde.org/D15295</a></div></div><br /><div><strong>To: </strong>rjvbb, Konsole, gateau, hindenburg<br /><strong>Cc: </strong>tcanabrava, konsole-devel, KDE Applications, herrold, ngraham, maximilianocuria, hindenburg<br /></div>