https://bugs.kde.org/show_bug.cgi?id=365375
David Faure <fa...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fa...@kde.org
--- Comment #3 from David Faure <fa...@kde.org> ---
This patch looks awful to me ;)
(it duplicates what KRun does much better, it breaks on files with a
double-quote in the name, and for non-local files, it breaks on any other
special command from the desktop entry spec like %d since it will keep it
instead of providing its value, etc.)
Why not just use KRun::runService() or (since KF 5.24) KRun::runApplication() ?
For the other overload (without a kservice), there's KRun::run().
--
You are receiving this mail because:
You are watching all bug changes.
