Hi Dominik,<br><br>thanks for your reply and for the link. For fixing the krazy2 issues correctly I changed QString to QLatin1String. I understand your point of view with &quot;readability&quot; but I won&#39;t decide which way is better. I think the explicit conversion of these Strings makes the understanding of the code not so much harder...<br>
<br>You can find the modified patch in the attachment. <br><br><div class="gmail_quote">2009/1/15 Dominik Haumann <span dir="ltr">&lt;<a href="mailto:dhdev@gmx.de">dhdev@gmx.de</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Philipp,<br>
<br>
On Thursday 15 January 2009, Philipp Klaffert wrote:<br>
&gt; @@ -54,7 +53,7 @@<br>
&gt; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;return true;<br>
&gt; &nbsp; &nbsp; &nbsp;}<br>
&gt;<br>
&gt; - &nbsp; &nbsp;if (!name.startsWith(&quot;Timeline:&quot;) &amp;&amp;<br>
&gt; !name.startsWith(&quot;TimelineWithFriends:&quot;) &nbsp;&amp;&amp;<br>
&gt; !name.startsWith(&quot;Profile:&quot;)) {<br>
&gt; + &nbsp; &nbsp;if<br>
&gt; (!name.startsWith(QString(&quot;Timeline:&quot;)) &amp;&amp;<br>
&gt; !name.startsWith(QString(&quot;TimelineWithFriends:&quot;)) &nbsp;&amp;&amp;<br>
&gt; !name.startsWith(QString(&quot;Profile:&quot;))) { return false;<br>
&gt; &nbsp; &nbsp; &nbsp;}<br>
<br>
This is also what implicitly happens: const char* is implicitely converted<br>
to a QString. The right way is afaik to use QLatin1String, see also<br>
<a href="http://doc.trolltech.com/latest/qlatin1string.html" target="_blank">http://doc.trolltech.com/latest/qlatin1string.html</a> for a better<br>
explanation :)<br>
<br>
And then, such optimizations should only be used in time critical cases, as<br>
the readability decreases when wrapping all strings (more text to read).<br>
But that&#39;s just my personal opinion, and maybe that&#39;s always the case in<br>
plasma - I don&#39;t know the preferred way here.<br>
<br>
Hope this helps :)<br>
<font color="#888888">Dominik<br>
</font></blockquote></div><br>