Review Request 127215: simplify code, reduce pointer dereferences
Nick Shaforostoff
shafff at ukr.net
Mon Feb 29 00:43:32 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127215/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kservice
Description
-------
Changes are mostly related to containers/iterators, but there are also few QString related optimizations
note that i have made a switch from QLinkedList to just QList because it performs better for T=<void*>
also i have changed QStringList to QSet<QString> in one place to make the code run faster
Diffs
-----
src/sycoca/kbuildmimetypefactory.cpp 340f76a
src/sycoca/kbuildservicefactory.cpp a74f2e8
src/sycoca/kbuildsycoca.cpp e99e906
src/sycoca/kmimeassociations.cpp 9423b27
src/sycoca/ksycocadict.cpp f33d01a
src/sycoca/ksycocafactory.cpp e410581
Diff: https://git.reviewboard.kde.org/r/127215/diff/
Testing
-------
all tests pass, except kmimeassociationstest (fakeApplicationService is NULL), but it fails also without my changes
Thanks,
Nick Shaforostoff
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160229/a244fe54/attachment.html>
More information about the Kde-frameworks-devel
mailing list