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

Reply via email to