----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128514/ -----------------------------------------------------------
(Updated July 31, 2016, 4:35 p.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks and Christoph Feck. Changes ------- Submitted with commit 757afa83ac74c51421cf16bcc44d53623bcfa908 by David Faure to branch master. Repository: kiconthemes Description ------- In the first pass, we already checked whether the icon exists in all icon themes. So in the second pass, there's no point in starting with that again, we can go ahead and start doing fallbacks right away. (The comment about the two passes was removed in b84858c, but some of it still applies.) Tested with: strace -e file ./kiconengine_unittest testUnknownIconNotCached |& wc -l Before: 6341 After: 4589 Diffs ----- autotests/kiconloader_benchmark.cpp ded5e0221f8126139a019a8310e8f120672ac3b4 src/kiconloader.cpp 951775d77b7ec0e2d8099ee192ca25321dab4544 Diff: https://git.reviewboard.kde.org/r/128514/diff/ Testing ------- strace as mentionned above. All unittests still pass (incl fallback lookups, didn't check how complete those were, though) Thanks, David Faure
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
