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