2012/2/21  <span dir="ltr"><<a href="mailto:jan-arve.saether@nokia.com">jan-arve.saether@nokia.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
return QString::asprintf("%d bytes", int(number));<br>
should be:<br>
return tr("%n bytes", 0, number);<br>
<br>
Same here:<br>
const QString num = QString::asprintf("%.1f KB/s", bytesPerSecond / 1024.0);<br>
should be:<br>
const QString num = tr("%1 KB/s").arg(bytesPerSecond/1024.0, 0, 'f', 1);<br>
<br>
...And the list goes on (I stopped at qmap.cpp)<br></blockquote><br>I don't know if this is really the right comparison, both are slower and harder to understand.<br><br>sprintf() is great for formatted output, much better than chaining .args(). It also has the potential for being a lot faster.<br>
<br>It sounds like a step in the wrong direction to remove QString::sprintf() and friends without adding something better.<br><br>The purpose of the function is, after all, to allow construction of formatted output without concatenating substrings.<br>
<br></div>-- <br>Andreas Aardal Hanssen<br>