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

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 can't

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:

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

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 can't

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, or can