dfaure added a comment.
Ah wait this could be made faster to avoid the double lookup. if (d->subjobs.removeAll(job) > 0) { job->setParent(nullptr); return true; } return false; (and then we don't even need the job==nullptr check, that's included in the removeAll check) REPOSITORY R244 KCoreAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D9004 To: jtamate, #frameworks, dfaure