----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108230/#review25051 -----------------------------------------------------------
This review has been submitted with commit 55b759ab8b9f2d45e486fc84495682f00d9326d0 by Weng Xuetian to branch KDE/4.10. - Commit Hook 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 > >