Fixed suggested comments. Local build compiles with no errors.<div><br></div><div>-Jignesh<br><br><div class="gmail_quote">On Mon, Jun 11, 2012 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 Monday 11 June 2012 00:34:58 Jignesh Kakadiya wrote:<br>
> Hi,<br>
><br>
> Attached is the patch which fixes below warning.<br>
><br>
> kinit.cpp:543:78: warning: ‘static QString KGlobalSettings::documentPath()’<br>
> is<br>
> deprecated (declared at /d/kde/src/5/kdelibs-<br>
> frameworks/staging/kwidgets/src/utils/kglobalsettings.h:228) [-Wdeprecated-<br>
> declarations]<br>
><br>
> Please review it.<br>
<br>
</div></div>Don't remove the old implementation, it will be needed by other KDE modules.<br>
<br>
I'm surprised you didn't have to #include <qstandardpaths.h> anywhere, but<br>
maybe that's because kconfig.h includes it indirectly. You could also check if<br>
you can remove #include <kglobalsettings.h> from any of the files you ported,<br>
in case it was the last occurence of KGlobalSettings in the file.<br>
<br>
Looks good, otherwise (assuming you checked that everything compiles, of<br>
course).<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>
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5<br>
<br>
</font></span></blockquote></div><br></div>