D5755: Expose PlasmaWindow::pid through WaylandTasksModel.

Eike Hein noreply at phabricator.kde.org
Mon May 8 09:57:48 UTC 2017


hein added a comment.


  Argh, the lack of sleep ... I updated the wrong Phab. Give me a few mins ;)

INLINE COMMENTS

> graesslin wrote in waylandtasksmodel.cpp:340
> You are aware that pid is not always correct? Any process started by KWin shares KWin pid. Given that one need to be careful outside of KWin. You should not base any decisions on it and especially not try to close apps based on pid.

Yeah, I'm aware based on how ClientConnection must work. I'll enhance the AppPid role apidox in the abstract model. It's fine with how it's being used currently.

REPOSITORY
  R108 KWin

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

To: hein, #plasma, davidedmundson, sebas
Cc: kwin, #kwin, graesslin, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170508/6300812b/attachment.html>


More information about the Plasma-devel mailing list