D13814: Speedup sort

Jaime Torres Amate noreply at phabricator.kde.org
Sat Jun 30 16:41:04 BST 2018


jtamate created this revision.
jtamate added reviewers: Dolphin, Frameworks.
Restricted Application added a project: Dolphin.
Restricted Application added a subscriber: kfm-devel.
jtamate requested review of this revision.

REVISION SUMMARY
  Use a lambda function instead of a class.
  This way the member QCollator is not copied and initialized several times.

TEST PLAN
  Sorting in a directory with 82874 images:
  [TIME] Sorting: 19883  (before)
  [TIME] Sorting: 4198 (after)
  
  kfileitemmodelbenchmark before: ..............   Passed   29.36 sec
  kfileitemmodelbenchmark after:    ..............   Passed   20.39 sec

REPOSITORY
  R318 Dolphin

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

AFFECTED FILES
  src/kitemviews/kfileitemmodel.cpp

To: jtamate, #dolphin, #frameworks
Cc: kfm-devel, spoorun, navarromorales, isidorov, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180630/187e9a82/attachment.htm>


More information about the kfm-devel mailing list