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

Reply via email to