D6579: KDBusService: pass executable name to QStringList arguments in the activateRequested signal

2017-07-09 Thread David Faure
dfaure abandoned this revision. dfaure added a comment. It was already documented that the arg list could be empty when called directly via DBus, so this fix is wrong. Apps have to handle empty args list correctly. Docu updated in

D6579: KDBusService: pass executable name to QStringList arguments in the activateRequested signal

2017-07-09 Thread David Faure
dfaure updated this revision to Diff 16374. dfaure added a comment. Make the behaviour explicit in the documentation. REPOSITORY R271 KDBusAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6579?vs=16373=16374 BRANCH master REVISION DETAIL

D6579: KDBusService: pass executable name to QStringList arguments in the activateRequested signal

2017-07-09 Thread David Faure
dfaure created this revision. Restricted Application added a project: Frameworks. REVISION SUMMARY It was done in one case (CommandLine) and not the other (Activate), making things very inconsistent. An empty list of arguments makes QCommandLineParser fail with a warning (or even exit, when