Review Request 128465: KIconLoader: massive speed improvement for loading unavailable icons

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128465/
-----------------------------------------------------------

(Updated July 18, 2016, 7:54 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, Christoph Feck, David Rosca, Michael Pyne, and Olivier Goffart.


Changes
-------

Submitted with commit 6d1abc297486725c891d14c7fd3169f4575ec86d by David Faure to branch master.


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
-----

  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/20160718/a54938d2/attachment-0001.html>