Review Request 124811: KIconLoader: speed up hasIcon/iconPath by using the on-disk cache from loadIcon

[David Faure]

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

(Updated July 13, 2016, 10:31 p.m.)


Status
------

This change has been discarded.


Review request for KDE Frameworks and Christoph Feck.


Repository: kiconthemes


Description
-------

This makes QIcon::fromTheme() much faster, for all cases where we have loaded the
icon once before (including in another process).


Diffs
-----

  autotests/kiconloader_unittest.cpp 6b60e7ebfc970c94ae865d56c4e33a8034b4fcee 
  src/kiconloader.cpp db3aa8f8fd6f8fb706edcb27cce073c36831934d 

Diff: https://git.reviewboard.kde.org/r/124811/diff/


Testing
-------

The unittest has a way to see if KIconLoader used the cache or searched on disk, with the newly exported global int. I couldn't think of a better way.

I'll let Volker do the real-world testing and measure the overall impact as he did previously.


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160713/5daadaef/attachment.html>