D4925: actually delete old theme caches

David Edmundson noreply at phabricator.kde.org
Tue Mar 7 13:12:33 UTC 2017


davidedmundson added inline comments.

INLINE COMMENTS

> theme_p.cpp:271
> +        for (const QFileInfo &file : cacheDir.entryInfoList()) {
> +            if (!file.absoluteFilePath().endsWith(svgElementsFileName)) {
> +                QFile::remove(file.absoluteFilePath());

do we not want

if (! file.fileName() ==

?

REPOSITORY
  R242 Plasma Framework (Library)

BRANCH
  arcpatch-D4925

REVISION DETAIL
  https://phabricator.kde.org/D4925

To: mart, #plasma, sitter
Cc: davidedmundson, sitter, 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/20170307/e85bb4ee/attachment.html>


More information about the Kde-frameworks-devel mailing list