----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122414/ -----------------------------------------------------------
(Updated Feb. 6, 2015, 9:22 nachm.) Review request for Solid and Martin Gräßlin. Changes ------- Remove xcb_flushes, works without them Repository: powerdevil Description ------- The documentation says that error events for void calls will be delivered to the event loop, what does that mean for us? Will Qt throw them away, or do we need to install an event handler for them similar to what the previous code did? Or use the checked variants? Also adds some C++11 niceties and overall coding style cleanup. DPMS is now a required dependency for PowerDevil (is this a bad thing? given the thing was explicitly split out of the other actions). Diffs (updated) ----- daemon/actions/dpms/powerdevildpmsaction.cpp 8586f94 CMakeLists.txt c9e9047 daemon/actions/CMakeLists.txt a1b70ca daemon/actions/dpms/CMakeLists.txt 79c4062 daemon/actions/dpms/powerdevildpmsaction.h 31e11c6 Diff: https://git.reviewboard.kde.org/r/122414/diff/ Testing ------- Power button can turn off the screen, and after timeout turns off and properly wakes up again. Did not test inhibition handling. Thanks, Kai Uwe Broulik
_______________________________________________ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel