]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Fix comment
authorPeter Penz <peter.penz19@gmail.com>
Tue, 13 Dec 2011 23:03:03 +0000 (00:03 +0100)
committerPeter Penz <peter.penz19@gmail.com>
Tue, 13 Dec 2011 23:03:40 +0000 (00:03 +0100)
src/kitemviews/kfileitemmodel.cpp

index d81a985c73605061317f0ecf2c0f7886465edae8..71439fc18d79d245978b80820a0ebc596de274fd 100644 (file)
@@ -1473,15 +1473,16 @@ int KFileItemModel::expansionLevelsCompare(const KFileItem& a, const KFileItem&
 
     // TODO: The following implementation is very hacky but works. Issues with
     // this approach:
-    // 1. m_items is accessed, although the sorting in might also work on
-    //    non-member variables. This happens in insertItems() but it does
+    // 1. m_items is accessed, although the sorting might also work on
+    //    non-member variables. This happens in insertItems() but it does
     //    not really matter as in this case only a presorting is done.
     // 2. It is very slow in theory as it introduces a O(n*n) runtime complexity
     //    in combination with lessThan(). Practically the code is still fast
     //    enough for thousands of items but this must be fixed.
     //
     // Proposal: Extend the internal structure ItemData by a member
-    // 'ItemData* parent and access it here.
+    // 'ItemData* parent' and access it here. It should be sufficient
+    // to update 'parent' in combination with the expansionLevel.
 
     const KUrl parentUrlA(pathA.left(index) + subPathA);
     const KUrl parentUrlB(pathB.left(index) + subPathB);