meven marked 2 inline comments as done.
meven added inline comments.

INLINE COMMENTS

> dfaure wrote in directorysizejob.cpp:137
> Now that I look at this again, we do NOT want to follow symlinks here.
> 
> This was a bug in the existing code.
> 
> In Dolphin we want to display the size of the target file (more useful 
> information than the size of the symlink itself), but when calculating 
> directory size, we do NOT want to follow the symlink, the target size does 
> not count.
> 
> These new flags allow to fix this long-standing bug.

The previous code did (details of 3)

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D25010

To: meven, #frameworks, dfaure, kossebau
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to