Review Request 125750: Reduce some allocations
Aleix Pol Gonzalez
aleixpol at kde.org
Thu Oct 22 01:52:43 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125750/
-----------------------------------------------------------
Review request for KDE Frameworks and Sebastian Kügler.
Repository: kcoreaddons
Description
-------
I know this could be done so much better and there's a comment saying it's transitional (heh), but this happens to be called a ton of times on every plasma boot through [1].
Actually, why is it transitional? Are we supposed to move from metadata.desktop to metadata.json? Or is it because we should all be using `kpackagetool5 --generate-index`?
[1] KPluginMetaData::loadFromDesktopFile(QString const&, QStringList const&)
Diffs
-----
src/lib/plugin/desktopfileparser.cpp 7a4556a
Diff: https://git.reviewboard.kde.org/r/125750/diff/
Testing
-------
Tests pass, plasma still boots.
Thanks,
Aleix Pol Gonzalez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151022/1ba69da6/attachment.html>
More information about the Kde-frameworks-devel
mailing list