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