D18356: [Plasma Theme] Use new connect syntax
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Jan 18 11:31:56 GMT 2019
broulik created this revision.
broulik added a reviewer: Plasma.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
broulik requested review of this revision.
REVISION SUMMARY
Saves some cycles
TEST PLAN
Saves like 0.6ms on startup here, given the amount of `Theme` objects created.
`ThemePrivate` is also a `QObject`, I don't understand why it checks for `qApp` or why it's not done in the `Private`'s constructor though..
REPOSITORY
R242 Plasma Framework (Library)
REVISION DETAIL
https://phabricator.kde.org/D18356
AFFECTED FILES
src/plasma/theme.cpp
To: broulik, #plasma
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190118/34db785e/attachment.html>
More information about the Kde-frameworks-devel
mailing list