<div dir="ltr"><div style>Hi David,</div><div><br></div>Here is the first patch which removes QGuiAppliction from kprotocolmanager.<div style>Please review it.</div><div style><br></div><div style>Thanks,</div><div style>Jignesh</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 11, 2013 at 10:45 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="im">On Monday 11 February 2013 21:59:44 Jignesh Kakadiya wrote:<br>
> Hi David,<br>
><br>
> It moves kprotocolmanager from kio/kio to kiocore. Builds successfully.<br>
> Please review it.<br>
<br>
</div>You should make two commits: one which moves the code, and one which makes<br>
changes to the code (like the QGuiApplication dependency thing).<br>
<br>
I don't mind if the commit that moves the code, makes simple adjustements like<br>
changing export macros, I've done that before (and my script does that).<br>
But other changes should be separate, so that they can be reviewed.<br>
<br>
On that note I think you got that bit wrong: you should get rid of the #if<br>
QT_VERSION blocks, and only keep the code you want to keep<br>
(which, in kprotocolmanager.cpp, is the code that doesn't use QGuiApplication)<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>