jtamate added a comment.

  In D10342#286023 <https://phabricator.kde.org/D10342#286023>, @hein wrote:
  
  > This patch doesn't make any sense. It's setting up a cache for something 
computed from data that's subject to change, and it's never evicting it when 
that data changes.
  
  
  To make it right, should it also be wiped in 
XWindowTasksModel::Private::windowChanged?
  Or perhaps is it better to try to create that cache in KServiceTypeTrader?

REPOSITORY
  R120 Plasma Workspace

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

To: jtamate, #plasma_workspaces, hein
Cc: mart, mwolff, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol

Reply via email to