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

List:       kde-commits
Subject:    [kdepim-runtime/KDE/4.13] agents/newmailnotifier: Minor optimization
From:       Montel Laurent <montel () kde ! org>
Date:       2014-03-23 13:13:20
Message-ID: E1WRiDI-000658-73 () scm ! kde ! org
[Download RAW message or body]

Git commit 8db602e19b1b05edced7b9b923ecfd42f134ce1f by Montel Laurent.
Committed on 23/03/2014 at 13:13.
Pushed by mlaurent into branch 'KDE/4.13'.

Minor optimization

M  +3    -3    agents/newmailnotifier/newmailnotifieragent.cpp

http://commits.kde.org/kdepim-runtime/8db602e19b1b05edced7b9b923ecfd42f134ce1f

diff --git a/agents/newmailnotifier/newmailnotifieragent.cpp \
b/agents/newmailnotifier/newmailnotifieragent.cpp index 3a4f31d..45fab40 100644
--- a/agents/newmailnotifier/newmailnotifieragent.cpp
+++ b/agents/newmailnotifier/newmailnotifieragent.cpp
@@ -285,7 +285,7 @@ void NewMailNotifierAgent::itemsRemoved(const Item::List &items )
     for ( QHash< Akonadi::Collection, QList<Akonadi::Item::Id> >::iterator it = \
mNewMails.begin(); it != end; ++it ) {  QList<Akonadi::Item::Id> idList = it.value();
         bool itemFound = false;
-        Q_FOREACH( Item item, items ) {
+        Q_FOREACH( const Item &item, items ) {
             if (idList.contains(item.id())) {
                 idList.removeAll( item.id() );
                 itemFound = true;
@@ -305,7 +305,7 @@ void NewMailNotifierAgent::itemsFlagsChanged( const \
Akonadi::Item::List &items,  {
     if (!isActive())
         return;
-    Q_FOREACH (Akonadi::Item item, items) {
+    Q_FOREACH (const Akonadi::Item &item, items) {
         QHash< Akonadi::Collection, QList<Akonadi::Item::Id> >::iterator \
                end(mNewMails.end());
         for ( QHash< Akonadi::Collection, QList<Akonadi::Item::Id> >::iterator it = \
mNewMails.begin(); it != end; ++it ) {  QList<Akonadi::Item::Id> idList= it.value();
@@ -327,7 +327,7 @@ void NewMailNotifierAgent::itemsMoved( const Akonadi::Item::List \
&items, const A  if (!isActive())
         return;
 
-    Q_FOREACH (Akonadi::Item item, items) {
+    Q_FOREACH (const Akonadi::Item &item, items) {
         Akonadi::MessageStatus status;
         status.setStatusFromFlags( item.flags() );
         if ( status.isRead() || status.isSpam() || status.isIgnored() )


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