----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115326/#review48530 -----------------------------------------------------------
This needs forward-porting to KF5/plasma-framework, though. - Sebastian Kügler On Jan. 28, 2014, 9:18 a.m., Harald Sitter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115326/ > ----------------------------------------------------------- > > (Updated Jan. 28, 2014, 9:18 a.m.) > > > Review request for Plasma, Aaron J. Seigo and Martin Klapetek. > > > Repository: kdelibs > > > Description > ------- > > - introduce svgelements cache handling in useCache() > - to decide whether or not to discard a cache type now the mtimes of > metadata.desktop and the respective cache file are compared (previously this > was an invalid file compared to cache change time) > - cache cleanup now happens entirely before creation of the KImageCache > instance to prevent any sort of problem there > > - add fixme comment about broken version-based cache handling. for one the > version specific file name is not actually used for the KImageCache and for > another the entire shebang is missing from svgelements all together. this has > the very unfortunate side effect that the *active* cache files are deleted > ever so often, it may be worth removing the code portions until someone finds > time/interest to actually implement this in a working fashion. > > > Diffs > ----- > > plasma/theme.cpp cb44878 > > Diff: https://git.reviewboard.kde.org/r/115326/diff/ > > > Testing > ------- > > upgraded from 4.8 straight to 4.12 -> theme caches correctly discarded > > > Thanks, > > Harald Sitter > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
