[Kde-hardware-devel] Review Request: Emit battery low/critical notification when starting the session

[Kai Uwe Broulik]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107117/
-----------------------------------------------------------

Review request for Solid.


Description
-------

When starting the system while the battery is already below the warning/critical battery level threshold, there will be no notification about that.
This patch fixes this by introducing a m_batteriesPercentNotified bool that is false on startup and if it is false, the battery low notification will be triggered if the battery is lower or becomes lower than the threshold no matter if it already has been lower.


This addresses bug 298604.
    http://bugs.kde.org/show_bug.cgi?id=298604


Diffs
-----

  powerdevil/daemon/powerdevilcore.h 7140051 
  powerdevil/daemon/powerdevilcore.cpp 3c40780 

Diff: http://git.reviewboard.kde.org/r/107117/diff/


Testing
-------

Set my low battery level to 90% and started the system. Once the desktop was up I got the notification that my battery was low.
I don't know if it is good to also start the suspend timer right when the desktop comes up, though.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20121029/a6e4dc9d/attachment.html>