Review Request 127763: Reduce memory usage footprint

Aleix Pol Gonzalez aleixpol at kde.org
Thu Apr 28 12:38:16 UTC 2016


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

(Updated April 28, 2016, 2:38 p.m.)


Review request for KDE Frameworks and Christoph Feck.


Changes
-------

Make Kai and Milian happy.


Repository: kiconthemes


Description
-------

Leverages the implicitly shared nature of QString by only constructing the paths when it's strictly necessary. It souldn't show a big performance impact since the generated paths were only intermediate in most cases after all.

This is especially important now that plasma has several KIconLoader instances at the same time, but it should have an impact on every application using kiconthemes (or under the plasma-integration effects).


Diffs (updated)
-----

  src/kicontheme.cpp 735ecf7 

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


Testing
-------

Tests still pass, apps still work.

Starting and closing dolphin under callgrind showed 1640M instructions, now 1466M(10% improvement).


File Attachments (updated)
----------------

after
  https://git.reviewboard.kde.org/media/uploaded/files/2016/04/28/3f9b64d7-58e0-454b-9774-6ec86aa7f0c3__heaptrack.dolphin.16572-after.gz


Thanks,

Aleix Pol Gonzalez

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


More information about the Kde-frameworks-devel mailing list