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: How to help?

2012-06-11 Thread Kevin Ottens
Hello, On Tuesday 22 May 2012 13:27:56 Paweł Paprota wrote: > I have been looking through KDE projects for some time now in order to > find some work. I have found minor stuff like bugfixing in plasmoids > etc. minor improvements. > > I also stumbled upon KDE frameworks - I really like this effor

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

Re: API review: KDBusApplicationStarter

2012-06-11 Thread Thiago Macieira
On terça-feira, 22 de maio de 2012 23.03.22, David Faure wrote: > I'm about to write this class for libkdbus > (or possibly for Qt, if thiago thinks this could go into libQtDBus?), > to replace KToolInvocation::startService*. Can't we use D-Bus service starting now? What is missing in the daemon?

Re: API review: KDBusApplicationStarter

2012-06-11 Thread Sune Vuorela
On 2012-05-22, David Faure wrote: > bool waitForReady(int msecs = 3); This one waits and has a timeout > static bool execute(const QString &program, const QString > &expectedServiceName, > const QStringList &arguments = QStringList(), QString > *error = 0);

Re: API review: KDBusApplicationStarter

2012-06-11 Thread Thiago Macieira
On quarta-feira, 23 de maio de 2012 10.18.26, David Faure wrote: > On Wednesday 23 May 2012 01:29:52 Thiago Macieira wrote: > > On terça-feira, 22 de maio de 2012 23.03.22, David Faure wrote: > > > I'm about to write this class for libkdbus > > > (or possibly for Qt, if thiago thinks this could go

How to help?

2012-06-11 Thread Paweł Paprota
Hello all, I have been looking through KDE projects for some time now in order to find some work. I have found minor stuff like bugfixing in plasmoids etc. minor improvements. I also stumbled upon KDE frameworks - I really like this effort to reduce the duplication, improve the overall architectu