----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107117/#review21120 -----------------------------------------------------------
TBH for this change, instead of having a bool hanging around, I'd instead force calling the onBatteryPercentChanged everytime a new battery is added. This also adds a neat side-effect: not only it makes the feature available for multi-batteries, but also whenever a new battery is added to the system. - Dario Freddi On Oct. 29, 2012, 4:13 p.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107117/ > ----------------------------------------------------------- > > (Updated Oct. 29, 2012, 4:13 p.m.) > > > 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 > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
