D4925: actually delete old theme caches
Aleix Pol Gonzalez
noreply at phabricator.kde.org
Fri Mar 3 16:40:53 UTC 2017
apol added inline comments.
INLINE COMMENTS
> theme_p.cpp:221
> +
> + for (const QString &file : cacheDir.entryList()) {
> if (currentCacheFileName.isEmpty() ||
Use entryInfoList? This way you won't have to construct the path again...
> theme_p.cpp:270
> +
> + for (const QString &file : cacheDir.entryList()) {
> + if (!file.endsWith(svgElementsFileName)) {
Same
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D4925
To: mart, #plasma
Cc: apol, plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170303/2585f910/attachment.html>
More information about the Kde-frameworks-devel
mailing list