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

List:       kde-commits
Subject:    edu.kde.org/developer
From:       Anne-Marie Mahfouf <annma () kde ! org>
Date:       2003-02-27 2:14:15
[Download RAW message or body]

CVS commit by annma: 


add a few tips about i18n inspired by antlarr commits
 


  M +20 -1     tips.phtml   1.7


--- edu.kde.org/developer/tips.phtml  #1.6:1.7
@@ -23,5 +23,24 @@
 </font>
 </p>
+Use correctly isEmpty() and isNull() for QStrings. Don't write<br />
+<font face="Courier New,Courier">
+if (myString == "")
+</font><br />
+but use instead<br />
+<font face="Courier New,Courier">
+if (myString.isEmpty())
+</font>
+</p>
+
+<p>i18n() is already a QString so don't write<br />
+<font face="Courier New,Courier">
+QString myString = QString(i18n("bla bla"));<br />
+</font>
+but use instead<br />
+<font face="Courier New,Courier">
+QString myString = i18n("bla bla");
+</font>
 </p>
+
 <p><a href="mailto:kevin.krammer@gmx.at">Kevin Krammer</a> has contributed some code to replace strings
 in a picture by the use of a painter that draws a message on a background picture.</p>


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