Fix DirectorySizeJob so it doesn't depend on listing order
ClosedPublic

Authored by dfaure on Apr 5 2020, 6:32 PM.

Details

Diff Detail

Repository
R241 KIO
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24809
Build 24827: arc lint + arc unit
dfaure created this revision.Apr 5 2020, 6:32 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 5 2020, 6:32 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
dfaure requested review of this revision.Apr 5 2020, 6:32 PM
ahmadsamir accepted this revision.Apr 5 2020, 7:27 PM

I am sure the test passed locally many times before I submitted D28472 (and it still passes).

But indeed it shouldn't depend on which was listed first, the dir or the link to it (retrospective wisdom on my part :D).

This revision is now accepted and ready to land.Apr 5 2020, 7:27 PM
dfaure closed this revision.Apr 5 2020, 10:03 PM