Review Request 122602: Get rid of brightness percentage
Kai Uwe Broulik
kde at privat.broulik.de
Tue Feb 17 13:35:49 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122602/
-----------------------------------------------------------
(Updated Feb. 17, 2015, 1:35 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma, Solid and Nikita Skovoroda.
Repository: powerdevil
Description
-------
This renames all the fooValue things to just foo, everywhere.
Set brightness no longer returns bool since we don't really check success anymore anyway.
Also adds a bit of C++11 sugar.
Diffs
-----
daemon/actions/bundled/brightnesscontrol.h a802174
daemon/actions/bundled/brightnesscontrol.cpp 84d4f66
daemon/actions/bundled/dimdisplay.h b4b0517
daemon/actions/bundled/dimdisplay.cpp ae578a8
daemon/actions/bundled/keyboardbrightnesscontrol.h cbbbc85
daemon/actions/bundled/keyboardbrightnesscontrol.cpp ee7449c
daemon/actions/bundled/org.kde.Solid.PowerManagement.Actions.BrightnessControl.xml 63f96f4
daemon/actions/bundled/org.kde.Solid.PowerManagement.Actions.KeyboardBrightnessControl.xml 777df2f
daemon/actions/dpms/powerdevildpmsaction.h c74b45b
daemon/actions/dpms/powerdevildpmsaction.cpp 42ff8f1
daemon/backends/hal/powerdevilhalbackend.h b34b893
daemon/backends/hal/powerdevilhalbackend.cpp 8904f52
daemon/backends/upower/backlighthelper.h cb53ec9
daemon/backends/upower/backlighthelper.cpp bf630fa
daemon/backends/upower/powerdevilupowerbackend.h 9c6c38f
daemon/backends/upower/powerdevilupowerbackend.cpp ff99137
daemon/backends/upower/xrandrbrightness.h d3a78b1
daemon/backends/upower/xrandrbrightness.cpp 493483b
daemon/powerdevilbackendinterface.h f5bc652
daemon/powerdevilbackendinterface.cpp 37d56f4
daemon/powerdevilbrightnesslogic.h 90b42b5
daemon/powerdevilbrightnesslogic.cpp 25c5de0
daemon/backends/upower/backlight_helper_actions.actions 14b714f
Diff: https://git.reviewboard.kde.org/r/122602/diff/
Testing
-------
Seems to work as before but needs more extensive testing
Thanks,
Kai Uwe Broulik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150217/477f7839/attachment.html>
More information about the Plasma-devel
mailing list