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