Compare UDS entry times directly instead of going through KFileItem

Authored by broulik on Jul 26 2018, 7:40 AM.



This avoids creating a QDateTime object with all the timezone processing that comes with it since we're only interested in the relative order, not absolute precise date time values.

Test Plan

Sort order seems identical to before, opening a folder with a gazillion files and sorting by modification time is 12x faster here.

Diff Detail

R318 Dolphin
Lint Skipped
Unit Tests Skipped
broulik created this revision.Jul 26 2018, 7:40 AM
Restricted Application added a project: Dolphin. · View Herald TranscriptJul 26 2018, 7:40 AM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
broulik requested review of this revision.Jul 26 2018, 7:40 AM

+2 from my part.
I was about to increment KFileItem to return that information also as long long.

dfaure accepted this revision.Jul 26 2018, 8:13 AM

Nice idea.

This revision is now accepted and ready to land.Jul 26 2018, 8:13 AM
This revision was automatically updated to reflect the committed changes.