mpyne accepted this revision. mpyne added a comment.
The diff as proposed is just fine. I've been bitten by `qDeleteAll` at proc exit before when called on objects that have complex destructors, but that's not the case here. It would probably be a good idea to try to streamline the code structurally so that we don't have to hold pointers. But given the difficulty that it would entail, I think it would deserve either a separate Differential review, or to "upgrade" this review to focus on the structural change for its own sake, instead of just working on the memleak at process exit. REPOSITORY R244 KCoreAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D4439 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: dfaure, aacid, mpyne Cc: markg, #frameworks
