dfaure updated this revision to Diff 18724. dfaure added a comment.
- Fix leak introduced by first version of the commit, we need to make a copy of the idle list before clearing it. - When deleting the slave because it emits a signal, we still need to use deleteLater(), I guess... The slaveDied() code path isn't unittested, which is why I didn't detect this earlier. REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D7529?vs=18722&id=18724 BRANCH master REVISION DETAIL https://phabricator.kde.org/D7529 AFFECTED FILES src/core/scheduler.cpp src/core/scheduler_p.h src/core/slave.cpp src/core/slave.h To: dfaure, mwolff Cc: #frameworks
