D24989: support multiple rcc files with icons themes
Hannah von Reth
noreply at phabricator.kde.org
Sun Oct 27 19:19:26 GMT 2019
vonreth requested changes to this revision.
vonreth added a subscriber: dfaure.
vonreth added a comment.
This revision now requires changes to proceed.
Rest looks good to me.
@dfaure you created the initial version what do you think?
INLINE COMMENTS
> kicontheme.cpp:79
> + for (const auto &iconDir : QStandardPaths::locateAll(QStandardPaths::AppDataLocation, QStringLiteral("icons"), QStandardPaths::LocateDirectory)) {
> + QDirIterator it(iconDir, QStringList() << QLatin1String("*.rcc"), QDir::Files);
> + while (it.hasNext()) {
I guess we can use an Initializer here
REPOSITORY
R302 KIconThemes
REVISION DETAIL
https://phabricator.kde.org/D24989
To: cullmann, #frameworks, vonreth
Cc: dfaure, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191027/497ddb11/attachment.html>
More information about the Kde-frameworks-devel
mailing list