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

Reply via email to