[Differential] [Commented On] D2967: delete containments upon activity deletion

[mart (Marco Martin)]

mart added a comment.


  In https://phabricator.kde.org/D2967#55135, @davidedmundson wrote:
  
  > > m_desktopContainments.remove(id);
  >
  > should not be here.
  
  
  why not? m_desktopContainments is a  QHash<QString, QSet<Plasma::Containment *> > indexed by activity, if i don't update it manually here, it will stay full of dangling stuff..

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, ivan, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161007/e8aeacf6/attachment.html>