> On April 5, 2015, 5:12 nachm., Lamarque Souza wrote: > > daemon/backends/upower/ddchelper.h, line 26 > > <https://git.reviewboard.kde.org/r/123262/diff/1/?file=360124#file360124line26> > > > > Please do not use "using" inside a header file. That can lead to wierd > > compiling errors.
This is done in the backlighthelper as well and without it the KAuth helper does not work at all for some reason. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123262/#review78529 ----------------------------------------------------------- On April 5, 2015, 3:44 nachm., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123262/ > ----------------------------------------------------------- > > (Updated April 5, 2015, 3:44 nachm.) > > > Review request for Solid. > > > Repository: powerdevil > > > Description > ------- > > This allows PowerDevil to use the ddccontrol utility to manipulate brightness > of external monitors through Display Data Channel. > > It becomes the last link in the fallback chain; it first tries XRandR, then > it tries the sysfs helper, and if all that fails it tries to use the > ddccontrol utility. > > dcccontrol unfortunately doesn't have machine-readable output so I have to > take apart its probing output to get the device name and id of the backlight > controls. Also it is pretty slow both initializing (kded startup, the helper > saves the values and writes directly to the address returned by the initial > probing) and setting brightness (that one doesn't block but it can take > seconds until your monitor actually changes brightness) > > > Diffs > ----- > > daemon/BackendConfig.cmake 295a8a2 > daemon/backends/upower/ddc_helper_actions.actions PRE-CREATION > daemon/backends/upower/ddchelper.h PRE-CREATION > daemon/backends/upower/ddchelper.cpp PRE-CREATION > daemon/backends/upower/powerdevilupowerbackend.h 1c4dd59 > daemon/backends/upower/powerdevilupowerbackend.cpp 87b9cc7 > > Diff: https://git.reviewboard.kde.org/r/123262/diff/ > > > Testing > ------- > > It's working pretty nicely, I can adjust brightness of my desktop monitor > through battery monitor and have it automatically dimmed after a timeout. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
