Review Request: optimization of kicontheme
Jaime Torres
jtamate at gmail.com
Mon Jun 29 17:42:21 BST 2009
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/908/
-----------------------------------------------------------
Review request for kdelibs.
Summary
-------
mBaseDir and mThemeDir are never used alone, and they were allways appened in dir() const.
Using callgrind I see that this simple patch reduces the time of KiconThemeDir::dir() const,
when kmail renders the mail from bug 186551, from taking 4,95% to only 0,34% (in almost 5 million calls).
Diffs
-----
/trunk/KDE/kdelibs/kdeui/icons/kicontheme.cpp 987707
Diff: http://reviewboard.kde.org/r/908/diff
Testing
-------
valgrind --tool=callgrind --log-file=kmail.valgrind.log --dump-after=KMReaderWin::parseMsg --zero-before=KMReaderWin::parseMsg --trace-children=yes /opt/kde4/bin/kmail --nofork
Thanks,
Jaime
More information about the kde-core-devel
mailing list