[patch] Setting the brightness on the command line

[Richard Hughes]

Try setting the brightness above range on the command line:

[hughsie at hughsie-laptop hal]$ dbus-send --system --print-reply
--dest=org.freedesktop.Hal /org/freedesktop/Hal/devices/acpi_lcd
org.freedesktop.Hal.Device.LaptopPanel.SetBrightness int32:8
Error org.freedesktop.Hal.Device.LaptopPanel.Invalid: Brightness has to
be between 0 and 8!

So the message is telling us to set a value between 0-8, which we really
are trying to do! The correct message would be to tell us the range was
0->7.

The attached patch fixes the message to:

[hughsie at hughsie-laptop hal]$ dbus-send --system --print-reply
--dest=org.freedesktop.Hal /org/freedesktop/Hal/devices/acpi_lcd
org.freedesktop.Hal.Device.LaptopPanel.SetBrightness int32:8
Error org.freedesktop.Hal.Device.LaptopPanel.Invalid: Brightness has to
be between 0 and 7

Okay to commit?

Richard.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-brightness-fix-error.patch
Type: text/x-patch
Size: 595 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20060828/84823f3e/hal-brightness-fix-error.bin