ahmadsamir created this revision.
ahmadsamir added reviewers: Frameworks, dfaure, meven.
Herald added a project: Frameworks.
ahmadsamir requested review of this revision.

REVISION SUMMARY
  After 730a6ddd828fb1fbdf0f3 
<https://phabricator.kde.org/R241:730a6ddd828fb1fbdf0f3d70412f6da4eeb1b912>, 
the behaviour changed and the code sets
  KIO::UDSEntry::UDS_DEVICE_ID after symlinks are resovled (if
  KIO::StatResolveSymlink is set); this meant that if we have dir A and
  a symlink to it B, the loop that detects hard-links will skip A if B
  was processed before, as both will have the same inode.
  
  This fixes the directorySize() unit test from kiocore-jobtest; the
  totalSubdirs() count didn't match the expected value.

TEST PLAN
  make && ctest

REPOSITORY
  R241 KIO

BRANCH
  l-dirsizejob (branched from master)

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

AFFECTED FILES
  src/core/directorysizejob.cpp

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

Reply via email to