D22471: [KProcessList] Split command line and process name
David Edmundson
noreply at phabricator.kde.org
Mon Jul 15 11:38:19 BST 2019
davidedmundson created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
davidedmundson requested review of this revision.
REVISION SUMMARY
Currently KProcessList had a method called "name" which confusingly
returns the entire command line.
In many cases where we want to use this class we need both. (See D22327 <https://phabricator.kde.org/D22327>)
This class exposes both "name" and "command" with the respective
properties.
In order to do this correctly we need to split the process name before
the null characters are replaced with spaces.
TEST PLAN
Unit test
Manually invoked ps list output
REPOSITORY
R244 KCoreAddons
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D22471
AFFECTED FILES
autotests/kprocesslisttest.cpp
src/lib/util/kprocesslist.cpp
src/lib/util/kprocesslist.h
src/lib/util/kprocesslist_p.h
src/lib/util/kprocesslist_unix.cpp
To: davidedmundson
Cc: kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190715/64910794/attachment.html>
More information about the Kde-frameworks-devel
mailing list