hal: Branch 'master'

Danny Kukawka dkukawka at kemper.freedesktop.org
Mon Mar 5 11:28:18 PST 2007


 hald/linux/addons/addon-dell-backlight.cpp |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

New commits:
diff-tree 7edc1f6587e7b5a0c463a738fc4508921f99bb94 (from e8275a44406339dfe5bd8550698c280bc6d8e173)
Author: Danny Kukawka <danny.kukawka at web.de>
Date:   Mon Mar 5 20:28:09 2007 +0100

    fixed compiler warning in addon-dell-backlight
    
    Fixed compiler warnings about comparison between signed and unsigned
    integer expressions.

diff --git a/hald/linux/addons/addon-dell-backlight.cpp b/hald/linux/addons/addon-dell-backlight.cpp
index 946e09b..39a73b6 100644
--- a/hald/linux/addons/addon-dell-backlight.cpp
+++ b/hald/linux/addons/addon-dell-backlight.cpp
@@ -144,7 +144,7 @@ filter_function (DBusConnection *connect
 					   &err,
 					   DBUS_TYPE_INT32, &brightness,
 					   DBUS_TYPE_INVALID)) {
-			if (brightness < minValue || brightness > maxValue) {
+			if (brightness < (int) minValue || brightness > (int) maxValue) {
 				reply = dbus_message_new_error (message,
 								"org.freedesktop.Hal.Device.LaptopPanel.Invalid",
 								"Brightness level is invalid");
@@ -178,9 +178,9 @@ filter_function (DBusConnection *connect
 					   DBUS_TYPE_INVALID)) {
 			int brightness = read_backlight (AC);
 
-			if (brightness < minValue)
+			if (brightness < (int) minValue)
 				brightness = minValue;
-			else if (brightness > maxValue)
+			else if (brightness > (int) maxValue)
 				brightness = maxValue;
 
 			reply = dbus_message_new_method_return (message);


More information about the hal-commit mailing list