Re: Patch: Fixes KGlobalSettings::documentPath() warnings.

2012-06-11 Thread David Faure
On Monday 11 June 2012 17:42:30 Jignesh Kakadiya wrote: > Fixed suggested comments. Local build compiles with no errors. Looks good, please commit. If you don't have an account yet, create an identity on identity.kde.org and then request developer priviledges. I forgot the details on how that wo

Re: Patch: Fixes KGlobalSettings::documentPath() warnings.

2012-06-11 Thread Jignesh Kakadiya
Fixed suggested comments. Local build compiles with no errors. -Jignesh On Mon, Jun 11, 2012 at 5:16 PM, David Faure wrote: > On Monday 11 June 2012 00:34:58 Jignesh Kakadiya wrote: > > Hi, > > > > Attached is the patch which fixes below warning. > > > > kinit.cpp:543:78: warning: ‘static QStri

Re: Patch: Fixes KGlobalSettings::documentPath() warnings.

2012-06-11 Thread David Faure
On Monday 11 June 2012 00:34:58 Jignesh Kakadiya wrote: > Hi, > > Attached is the patch which fixes below warning. > > kinit.cpp:543:78: warning: ‘static QString KGlobalSettings::documentPath()’ > is > deprecated (declared at /d/kde/src/5/kdelibs- > frameworks/staging/kwidgets/src/utils/kglobalse

Patch: Fixes KGlobalSettings::documentPath() warnings.

2012-06-11 Thread Jignesh Kakadiya
Hi, Attached is the patch which fixes below warning. kinit.cpp:543:78: warning: ‘static QString KGlobalSettings::documentPath()’ is deprecated (declared at /d/kde/src/5/kdelibs- frameworks/staging/kwidgets/src/utils/kglobalsettings.h:228) [-Wdeprecated- declarations] Please review it. Thanks, J