D28515: [CommandLauncherJob] Add constructor taking an executable and argument list
David Faure
noreply at phabricator.kde.org
Thu Apr 2 17:16:45 BST 2020
dfaure added a comment.
Good question. I haven't seen that done in most callers, but indeed, what if it's in a path with a space, like often happens on Windows?
It sounds like the answer is yes, it should be quoted.
The real answer is to write a CommandLauncherJob unittest for such a case, of course :-)
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D28515
To: broulik, #frameworks, dfaure
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200402/cd4f0d53/attachment.html>
More information about the Kde-frameworks-devel
mailing list