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