Changeset 13720 for trunk/qt/torrent-filter.cc

Timestamp:

Dec 30, 2012, 1:28:28 AM (9 years ago)

Author:

jordan

Message:

(qt) small speedup when rebuilding the activity combobox's counts: walk the torrent model just once, rather than once per activity

File:

• trunk/qt/torrent-filter.cc (modified) (2 diffs)

trunk/qt/torrent-filter.cc

@@ -11 +11 @@
  */
 
+#include <algorithm>
 #include <iostream>
 
@@ -206 +207 @@
 }
 
-int
-TorrentFilter :: count( const FilterMode& mode ) const
-{
-    int count = 0;
-
-    for( int row=0; ; ++row ) {
-        QModelIndex index = sourceModel()->index( row, 0 );
-        if( !index.isValid( ) )
-            break;
-        const Torrent * tor = index.data( TorrentModel::TorrentRole ).value<const Torrent*>();
-        if( activityFilterAcceptsTorrent( tor, mode ) )
-            ++count;
-    }
-
-    return count;
-}
+void
+TorrentFilter :: countTorrentsPerMode (int * setmeCounts) const
+{
+  std::fill_n (setmeCounts, FilterMode::NUM_MODES, 0);
+
+  for (int row(0); ; ++row)
+    { 
+      QModelIndex index (sourceModel()->index(row, 0));
+      if (!index.isValid())
+        break;
+
+      const Torrent * tor (index.data( TorrentModel::TorrentRole ).value<const Torrent*>());
+      for (int mode(0); mode<FilterMode::NUM_MODES; ++mode)
+        if (activityFilterAcceptsTorrent (tor, mode))
+          ++setmeCounts[mode];
+    }
+}
+