D10297: Add new "Tools" button above System Monitor's process list

Wed Mar 21 22:27:27 UTC 2018

rkflx added inline comments.

INLINE COMMENTS

> rkflx wrote in ksysguardprocesslist.cpp:412
> Could you do an exact match on the filename, i.e. only the last part of the full path? There might be situations where "System Monitor" is developed or installed in a directory containing this string by chance.

Recently I learned comparing with `qApp->desktopFileName()` might be even better than looking at the path of the executable…

REPOSITORY
  R111 KSysguard Library

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

To: gregormi, #plasma, colomar, kossebau, broulik, mart, hein
Cc: apol, anthonyfieroni, andreaska, rkflx, ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180321/80b155dd/attachment.html>