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