Review Request 121915: Add lidClosedChanged signal to org.kde.Solid.PowerManagement
Kai Uwe Broulik
kde at privat.broulik.de
Tue Jan 20 08:11:54 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121915/#review74374
-----------------------------------------------------------
This has been submitted, right?
- Kai Uwe Broulik
On Jan. 8, 2015, 12:04 nachm., Daniel Vrátil wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121915/
> -----------------------------------------------------------
>
> (Updated Jan. 8, 2015, 12:04 nachm.)
>
>
> Review request for Plasma and Solid.
>
>
> Repository: powerdevil
>
>
> Description
> -------
>
> There's no way to detect when lid has been closed other than listening to `changed` signal org.freedesktop.UPower and then polling the Powerdevil for new values. This patch adds a new signal to the PowerDevil interface to notify about the change and provide new value right away. Makes it much easier to use.
>
>
> Diffs
> -----
>
> daemon/org.kde.Solid.PowerManagement.xml 53f77e5
> daemon/powerdevilbackendinterface.h 702b66b
> daemon/powerdevilbackendinterface.cpp 6dd8c71
> daemon/powerdevilcore.h e255703
> daemon/powerdevilcore.cpp d51ab19
>
> Diff: https://git.reviewboard.kde.org/r/121915/diff/
>
>
> Testing
> -------
>
> Tested with qdbus-monitor, signal is emitted when laptop lid is closed/opened.
>
>
> Thanks,
>
> Daniel Vrátil
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150120/e67f4fce/attachment.html>
More information about the Plasma-devel
mailing list