D16875: [kded] Cleanup KScreenDaemon class

David Edmundson noreply at phabricator.kde.org
Wed Nov 14 11:36:43 GMT 2018


davidedmundson requested changes to this revision.
davidedmundson added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> daemon.h:47
> +    // DBus
> +    void applyLayoutPreset(const QString &presetName);
> +

this has moved from being a slot.

If it is invoked from DBus as the comment suggests that will be a problem

REPOSITORY
  R104 KScreen

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

To: romangg, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181114/fc192430/attachment-0001.html>


More information about the Plasma-devel mailing list