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