D15599: Port the "Switch Desktop" containment action to libtaskmanager

[Vlad Zagorodniy]

zzag added inline comments.

INLINE COMMENTS

> zzag wrote in desktop.cpp:73
> Will this loop ever be executed?
> 
> Also, why not `delete m_actions.take(i - 1);`?

Could someone please explain why this part is fine?

REPOSITORY
  R120 Plasma Workspace

BRANCH
  arcpatch-D15599

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

To: hein, mart, davidedmundson
Cc: zzag, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181016/1cb3c6c5/attachment.html>