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

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

SVN commit 1005903 by aacid:

const & in foreach


 M  +1 -1      dataengines/ocs/lib/ocsapi.cpp  
 M  +1 -1      dataengines/ocs/lib/postjob.cpp  
 M  +1 -1      dataengines/ocs/ocsengine.cpp  
 M  +1 -1      libs/rtm/session_p.h  


--- trunk/KDE/kdeplasma-addons/dataengines/ocs/lib/ocsapi.cpp #1005902:1005903
@@ -160,7 +160,7 @@
   KUrl url = createUrl( "content/data" );
 
   QStringList categoryIds;
-  foreach( Category category, categories ) {
+  foreach( const Category &category, categories ) {
     categoryIds.append( category.id() );
   }
   url.addQueryItem( "categories", categoryIds.join( "x" ) );
--- trunk/KDE/kdeplasma-addons/dataengines/ocs/lib/postjob.cpp #1005902:1005903
@@ -63,7 +63,7 @@
 {
   QString postData;
 
-  foreach( QString name, m_data.keys() ) {
+  foreach( const QString &name, m_data.keys() ) {
     m_url.addQueryItem( name, m_data.value( name ) );
   }
 
--- trunk/KDE/kdeplasma-addons/dataengines/ocs/ocsengine.cpp #1005902:1005903
@@ -318,7 +318,7 @@
             m_knowledgeBaseListJobs.remove(job);
         }
 
-        foreach (KnowledgeBase k, job->knowledgeBaseList()) {
+        foreach (const KnowledgeBase &k, job->knowledgeBaseList()) {
             setKnowledgeBaseData(source, k);
         }
         scheduleSourcesUpdated();
--- trunk/KDE/kdeplasma-addons/libs/rtm/session_p.h #1005902:1005903
@@ -85,7 +85,7 @@
   void smartListReply(RTM::Request* reply) {
     QStringList parts = reply->requestUrl().split("&");
     RTM::ListId id = 0;
-    foreach(QString part, parts)
+    foreach(const QString &part, parts)
       if (part.contains("rtm_internal_list_id"))
         id = part.split("=").last().toLongLong();
       
[prev in list] [next in list] [prev in thread] [next in thread]