----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122151/#review74346 -----------------------------------------------------------
Ship it! I don't see how this could make any difference but then I don't see how this could harm. And if it fixes the issue for you ... thanks! daemon/backends/upower/powerdevilupowerbackend.cpp <https://git.reviewboard.kde.org/r/122151/#comment51591> If you're already at, could you fix the coding style here? :) daemon/backends/upower/powerdevilupowerbackend.cpp <https://git.reviewboard.kde.org/r/122151/#comment51592> Same here - Kai Uwe Broulik On Jan. 19, 2015, 3:46 nachm., Hrvoje Senjan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122151/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2015, 3:46 nachm.) > > > Review request for Solid and Kai Uwe Broulik. > > > Bugs: 342597 > https://bugs.kde.org/show_bug.cgi?id=342597 > > > Repository: powerdevil > > > Description > ------- > > docs state the failure of exec shall be checked first. > > > Diffs > ----- > > daemon/backends/upower/powerdevilupowerbackend.cpp 51a052a > > Diff: https://git.reviewboard.kde.org/r/122151/diff/ > > > Testing > ------- > > resolves the bug for me (i have the brightness control in the battery applet > now) > > > Thanks, > > Hrvoje Senjan > >
_______________________________________________ Kde-hardware-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-hardware-devel
