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