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

List:       kde-commits
Subject:    branches/KDE/3.5/kdepim
From:       Carsten Wolff <wolff () kde ! org>
Date:       2008-05-07 18:08:31
Message-ID: 1210183711.070774.22083.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 805144 by wolff:

use a different separator for the acl list, as discussed in:
http://lists.kde.org/?l=kde-pim&m=121015531820742&w=2


 M  +3 -3      kioslaves/imap4/imap4.cc  
 M  +1 -1      kmail/acljobs.cpp  


--- branches/KDE/3.5/kdepim/kioslaves/imap4/imap4.cc #805143:805144
@@ -1493,10 +1493,10 @@
     }
     // Returning information to the application from a special() command isn't easy.
     // I'm reusing the infoMessage trick seen above (for capabilities), but this
-    // limits me to a string instead of a stringlist. I'm using space as separator,
-    // since I don't think it can be used in login names.
+    // limits me to a string instead of a stringlist. Using DQUOTE as separator,
+    // because it's forbidden in userids by rfc3501
     kdDebug(7116) << getResults() << endl;
-    infoMessage(getResults().join( " " ));
+    infoMessage(getResults().join( "\"" ));
     finished();
     break;
   }
--- branches/KDE/3.5/kdepim/kmail/acljobs.cpp #805143:805144
@@ -175,7 +175,7 @@
 void ACLJobs::GetACLJob::slotInfoMessage( KIO::Job*, const QString& str )
 {
   // Parse the result
-  QStringList lst = QStringList::split( " ", str, true );
+  QStringList lst = QStringList::split( "\"", str, true );
   while ( lst.count() >= 2 ) // we take items 2 by 2
   {
     QString user = lst.front(); lst.pop_front();
[prev in list] [next in list] [prev in thread] [next in thread]