This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:31c75e48c833: Read BAMF_DESKTOP_FILE_HINT from
environment (authored by broulik).
CHANGED PRIOR TO COMMIT
broulik added a comment.
`/proc` isn't disk io
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D22506
To: broulik, #plasma, hein
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel,
ZrenBot, himcesjf, lesliezhai, ali-mohamed,
hein added a comment.
The idea and approach are good.
But disk I/O in a UI hotpath is pretty scary. I do think a cache would be a
good addition.
In general I think libtm could use some improvements on its currently very
coarse cache eviction scheme. It's mostly fine but there are
broulik created this revision.
broulik added reviewers: Plasma, hein.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
This contains the actual name of the desktop file which might be different
from the one the