D22471: [KProcessList] Split command line and process name

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Jul 15 14:19:49 BST 2019


apol added a comment.


  So now name() will often return an empty string?

INLINE COMMENTS

> kprocesslist_unix.cpp:142
> +                int processNameStart = cmd.lastIndexOf('/', zeroIndex);
> +                if (processNameStart == -1) {
> +                    processNameStart = 0;

if it's -1, when you do ++ it will become 0 anyway.

REPOSITORY
  R244 KCoreAddons

REVISION DETAIL
  https://phabricator.kde.org/D22471

To: davidedmundson
Cc: apol, broulik, 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/b53f1ace/attachment.html>


More information about the Kde-frameworks-devel mailing list