<div dir="ltr">Here cmd is int so I haven't made any changes. for listit I used qWarning() followed by abort(). Please review it.<br><div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 9, 2013 at 5:16 PM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wednesday 09 January 2013 16:45:08 Jignesh Kakadiya wrote:<br>
> Hi David,<br>
><br>
> I have ported all kError calls to qWarning and kFatal to qFatal in<br>
> kdelibs/kio/kio directory. Compiles successfully. Please review it.<br>
><br>
> Next patch will be removal of kdebug.h include from kio/kio.<br>
<br>
</div></div>qFatal("Fatal Error: Got cmd %s, while waiting for an answer!", cmd);<br>
<br>
I doubt that cmd is a char*. If it's a QString, use qPrintable(cmd), if it's a<br>
QByteArray, use cmd.constData().<br>
Otherwise it compiles, but with a compiler warning (cannot pass a POD to<br>
"...") and it will crash at runtime (OK, for qFatal we expect a crash, but it<br>
helps if we can see the message first :-)<br>
<br>
<br>
+        qFatal() << "HUH? Lister" << listit << "is supposed to be listing,<br>
but has no job!";<br>
<br>
This one won't compile. You didn't notice because it's inside #if 0, but it<br>
should be fixed for whenever a developer re-enables that code. If qFatal is too<br>
cumbersome to use, you could use qWarning() followed by abort, for instance.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Working on KDE, in particular KDE Frameworks 5<br>
<br>
</font></span></blockquote></div><br></div></div></div></div></div>