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

List:       kde-commits
Subject:    KDE/kdelibs/kdecore/services
From:       Peter Kümmel <syntheticpp () yahoo ! com>
Date:       2009-11-13 9:55:37
Message-ID: 1258106137.193691.3794.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 1048394 by kuemmel:

don't stop at first invalid entry

 M  +12 -8     kservicetypeprofile.cpp  


--- trunk/KDE/kdelibs/kdecore/services/kservicetypeprofile.cpp #1048393:1048394
@@ -195,16 +195,20 @@
     config.writeEntry( "NumberOfEntries", count + disabledServices.count() );
     KService::List::ConstIterator servit = services.begin();
     int i = 0;
-    for( ; servit != services.end() && !servit->isNull(); ++servit, ++i ) {
-        const QString num = QString::number(i);
-        config.writeEntry( "Entry" + num + "_Service", (*servit)->storageId() );
-        config.writeEntry( "Entry" + num + "_Preference", count - i );
+    for( ; servit != services.end(); ++servit, ++i ) {
+        if (!servit->isNull()) {
+            const QString num = QString::number(i);
+            config.writeEntry( "Entry" + num + "_Service", (*servit)->storageId() );
+            config.writeEntry( "Entry" + num + "_Preference", count - i );
+        }
     }
     servit = disabledServices.begin();
-    for( ; servit != disabledServices.end() && !servit->isNull(); ++servit, ++i ) {
-        const QString num = QString::number(i);
-        config.writeEntry( "Entry" + num + "_Service", (*servit)->storageId() );
-        config.writeEntry( "Entry" + num + "_Preference", 0 );
+    for( ; servit != disabledServices.end(); ++servit, ++i ) {
+        if (!servit->isNull()) {
+            const QString num = QString::number(i);
+            config.writeEntry( "Entry" + num + "_Service", (*servit)->storageId() );
+            config.writeEntry( "Entry" + num + "_Preference", 0 );
+        }
     }
     configFile.sync();
 
[prev in list] [next in list] [prev in thread] [next in thread]