----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108230/#review24908 -----------------------------------------------------------
Ship it! Ship It! - Lukáš Tinkl On Jan. 6, 2013, 6 p.m., Xuetian Weng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108230/ > ----------------------------------------------------------- > > (Updated Jan. 6, 2013, 6 p.m.) > > > Review request for kde-workspace and Kai Uwe Broulik. > > > Description > ------- > > fix is trivial, since -1 is the invalid value in this part of code, not zero. > > Uncheck the brightness will set m_defaultValue to -1 so no function is lost. > > > This addresses bug 306925. > http://bugs.kde.org/show_bug.cgi?id=306925 > > > Diffs > ----- > > powerdevil/daemon/actions/bundled/brightnesscontrol.cpp 5088e4e > > Diff: http://git.reviewboard.kde.org/r/108230/diff/ > > > Testing > ------- > > now zero in profile works. > > > Thanks, > > Xuetian Weng > >