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