Re: Review Request 122149: Fix QCommandLineParser warnings in kcmshell5

2015-04-01 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122149/ --- (Updated April 1, 2015, 4:16 p.m.) Status -- This change has been di

Re: Review Request 122149: Fix QCommandLineParser warnings in kcmshell5

2015-01-19 Thread Aleix Pol Gonzalez
> On Jan. 19, 2015, 6:08 p.m., David Faure wrote: > > That's one option, and a good one for compat. > > > > The other option is to change the callers to pass a single dash instead, > > then Qt will process such options automatically - right? > > Aleix Pol Gonzalez wrote: > Yes, but then we

Re: Review Request 122149: Fix QCommandLineParser warnings in kcmshell5

2015-01-19 Thread David Faure
> On Jan. 19, 2015, 5:08 p.m., David Faure wrote: > > That's one option, and a good one for compat. > > > > The other option is to change the callers to pass a single dash instead, > > then Qt will process such options automatically - right? > > Aleix Pol Gonzalez wrote: > Yes, but then we

Re: Review Request 122149: Fix QCommandLineParser warnings in kcmshell5

2015-01-19 Thread Aleix Pol Gonzalez
> On Jan. 19, 2015, 6:08 p.m., David Faure wrote: > > That's one option, and a good one for compat. > > > > The other option is to change the callers to pass a single dash instead, > > then Qt will process such options automatically - right? Yes, but then we can't integrate them in our code, o

Re: Review Request 122149: Fix QCommandLineParser warnings in kcmshell5

2015-01-19 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122149/#review74347 --- That's one option, and a good one for compat. The other optio

Review Request 122149: Fix QCommandLineParser warnings in kcmshell5

2015-01-19 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122149/ --- Review request for KDE Frameworks and David Faure. Repository: kde-cli-to