From: Frank Reininghaus <frank78ac@googlemail.com>
Date: Tue, 5 Nov 2013 07:44:31 +0000 (+0100)
Subject: Only do a fast pre-sorting when "Sort by Name" is used
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/971aa6766576b3ca54e2b8539a396fe54f676155

Only do a fast pre-sorting when "Sort by Name" is used

This is a follow-up to commit 0e9f4a398735cfc19ae783d2ab054d2400d95416,
which tries to speed up sorting the items naturally by their name using
the idea that a fast non-natural pre-sorting already sorts the items
mostly correctly and thus reduces the number of expensive natural
comparisons.

This change only makes sense if the view is really sorted by "Name". In
other cases, the pre-sorting will most likely not be useful.

Thanks to Christoph Feck for pointing this out!
---

diff --git a/src/kitemviews/kfileitemmodel.cpp b/src/kitemviews/kfileitemmodel.cpp
index 261b23046..4c8577543 100644
--- a/src/kitemviews/kfileitemmodel.cpp
+++ b/src/kitemviews/kfileitemmodel.cpp
@@ -1034,7 +1034,7 @@ void KFileItemModel::insertItems(QList<ItemData*>& newItems)
 
     m_groups.clear();
 
-    if (m_naturalSorting) {
+    if (m_sortRole == NameRole && m_naturalSorting) {
         // Natural sorting of items can be very slow. However, it becomes much
         // faster if the input sequence is already mostly sorted. Therefore, we
         // first sort 'newItems' according to the QStrings returned by