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

Reply via email to