meven added inline comments. INLINE COMMENTS
> deletejob.cpp:130 > + /// Callback of worker rmfile > + void rmFileResult(bool result, bool isLink); > + /// Callback of worker rmdir Now we are at it, isLink could become m_curentURLIsLink, but isLink being a bool, it is much cheaper to move around threads, so we don't have perf incentive to do it as we had with a QString. REPOSITORY R241 KIO BRANCH arcpatch-D24962 REVISION DETAIL https://phabricator.kde.org/D24962 To: meven, dfaure, #frameworks Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns