D7879: [KConfigGroup] reserve() more and add some C++11

Kai Uwe Broulik noreply at phabricator.kde.org
Tue Sep 19 14:08:36 UTC 2017


broulik created this revision.
broulik added reviewers: kde-frameworks-devel, dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.

REVISION SUMMARY
  Using initializer_lists for QList we reserve the right amount of memory in advance and also make for nicer code.
  Also uses range-for where code as touched and a const container was used.

TEST PLAN
  Test still pass.
  
  `asRealList` is called 4200x on plasmashell startup for me, saves 2ms for me

REPOSITORY
  R237 KConfig

REVISION DETAIL
  https://phabricator.kde.org/D7879

AFFECTED FILES
  src/core/kconfiggroup.cpp

To: broulik, kde-frameworks-devel, dfaure
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170919/dc318d0d/attachment.html>


More information about the Kde-frameworks-devel mailing list