Review Request 128465: KIconLoader: massive speed improvement for loading unavailable icons
David Faure
faure at kde.org
Sun Jul 17 20:14:02 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128465/
-----------------------------------------------------------
(Updated July 17, 2016, 8:14 p.m.)
Review request for KDE Frameworks, Christoph Feck, David Rosca, Michael Pyne, and Olivier Goffart.
Changes
-------
Merge into a single hash, and avoid double lookups.
Repository: kiconthemes
Description
-------
Summary:
The code said "unknown icons should be searched for anew each time
[so that installing new icons works]". However this leads to massive
performance issues when opening the file dialog in a dir with many
files for which there is no mimetype icon.
In my case, 293 callgrind.out.* files in one dir (it's ironic that
they would create a huge performance issue...).
To make both sides happy (installing new icons should still work, but
still unknown icons shouldn't lead to a full disk search every time
icon.isNull() or icon.pixmap() is called), let's re-resolve unknown
icons again only after 5 seconds.
Benchmark results for loading an unavailable icon :
before: 43 msecs per iteration (1897 disk locations checked)
after: 0.013 msecs per iteration (pixmap found in the on-disk cache)
And the file dialog no longer crawls to a halt in the dir with 293 callgrind files.
Test Plan:
Reviewers:
Subscribers:
Diffs (updated)
-----
autotests/kiconengine_unittest.cpp 105e86c1e7bc6170c626aa80d34cdb6422acca9c
src/kiconloader.cpp d885318c166f2a996b038218366317615886a14e
Diff: https://git.reviewboard.kde.org/r/128465/diff/
Testing
-------
(described in commit log)
Thanks,
David Faure
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160717/0009036d/attachment.html>
More information about the Kde-frameworks-devel
mailing list