[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdeplasma-addons/applets
From:       Albert Astals Cid <tsdgeos () terra ! es>
Date:       2009-08-02 15:01:14
Message-ID: 1249225274.007488.14706.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 1005905 by aacid:

foreach already iterates over values so calling values explicitely is a waste both in \
memory and processing time


 M  +1 -1      lancelot/app/src/models/OpenDocuments.cpp  
 M  +1 -1      rssnow/news.cpp  


--- trunk/KDE/kdeplasma-addons/applets/lancelot/app/src/models/OpenDocuments.cpp \
#1005904:1005905 @@ -67,7 +67,7 @@
 
 void OpenDocuments::load()
 {
-    foreach (TaskPtr task, TaskManager::TaskManager::self()->tasks().values()) {
+    foreach (TaskPtr task, TaskManager::TaskManager::self()->tasks()) {
         if (setDataForTask(task)) {
             connectTask(task);
         }
--- trunk/KDE/kdeplasma-addons/applets/rssnow/news.cpp #1005904:1005905
@@ -139,7 +139,7 @@
         feedsUi.feedComboBox->addItem(name);
     }
 
-    foreach (const QString& name, akregatorFeeds().values()) {
+    foreach (const QString& name, akregatorFeeds()) {
         feedsUi.feedComboBox->addItem(name);
     }
 


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic