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

List:       kde-commits
Subject:    KDE/kdepimlibs/syndication
From:       Allen Winter <winter () kde ! org>
Date:       2008-12-31 23:27:58
Message-ID: 1230766078.981616.17957.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 903972 by winterz:

micro-optimization: .replace("X", "foo") -> .replace('X', "foo")


 M  +11 -11    tools.cpp  


--- trunk/KDE/kdepimlibs/syndication/tools.cpp #903971:903972
@@ -115,29 +115,29 @@
 QString escapeSpecialCharacters(const QString& strp)
 {
     QString str(strp);
-    str.replace("&", "&amp;");
-    str.replace("\"", "&quot;");
-    str.replace("<", "&lt;");
-    str.replace(">", "&gt;");
-    str.replace("\'", "&apos;");
+    str.replace('&', "&amp;");
+    str.replace('\"', "&quot;");
+    str.replace('<', "&lt;");
+    str.replace('>', "&gt;");
+    str.replace('\'', "&apos;");
     return str.trimmed();
 }
 
 QString convertNewlines(const QString& strp)
 {
     QString str(strp);
-    str.replace("\n", "<br/>");
+    str.replace('\n', "<br/>");
     return str;
 }
 
 QString plainTextToHtml(const QString& plainText)
 {
     QString str(plainText);
-    str.replace("&", "&amp;");
-    str.replace("\"", "&quot;");
-    str.replace("<", "&lt;");
-    //str.replace(">", "&gt;");
-    str.replace("\n", "<br/>");
+    str.replace('&', "&amp;");
+    str.replace('\"', "&quot;");
+    str.replace('<', "&lt;");
+    //str.replace('>', "&gt;");
+    str.replace('\n', "<br/>");
     return str.trimmed();
 }
 
[prev in list] [next in list] [prev in thread] [next in thread] 

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