D7510: [Service] Use KConfig::SimpleConfig

Kai Uwe Broulik noreply at phabricator.kde.org
Thu Aug 24 12:56:49 UTC 2017


broulik created this revision.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  I don't get the comment on why it is `/dev/null` instead of `QString()` but using `SimpleConfig` at least saves 0.3ms per call for me.

TEST PLAN
  Played with media controller (which uses service jobs to control the player), still works

REPOSITORY
  R242 Plasma Framework (Library)

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

AFFECTED FILES
  src/plasma/service.cpp

To: broulik, #plasma
Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170824/fd52537f/attachment.html>


More information about the Kde-frameworks-devel mailing list