| broulik |
| Plasma |
qDeleteAll will iterate through containments() however this is
dynamically changing the list we're iterating through.
Particularly bad is deleting the panel might delete the system tray
containment, leading to a double delete on a now dangly pointer.
kquitapp5 plasmashell. No crash
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
FWIW, longer term I believe the correct fix is to remove all of this.
Corona::~Corona will delete containments anyway (as the parent) and we already delete a view if a containment gets deleted. (or at least should) but I'm not going to make a change that big this close to release.