hal ChangeLog,1.280,1.281

Joe Shaw joe at freedesktop.org
Thu Sep 2 14:45:51 PDT 2004


Update of /cvs/hal/hal
In directory gabe:/tmp/cvs-serv12998

Modified Files:
	ChangeLog 
Log Message:
2004-09-02  Joe Shaw  <joeshaw at novell.com>

	* hald/device.c (hal_device_property_get_bool): Aiee!  If the
	property is not found, we want to return FALSE instead of -1.

	* hald/property.c (hal_property_get_bool): Return FALSE if the
	type isn't boolean in the assertion, not -1.

	* hald/linux/common.c (rename_and_merge): Fix some broken logic
	which only worked because it was depending on -1 being returned
	from boolean functions.  If I understand its original intent
	correctly, it should still work.  (Of course, info.not_available
	is never set to TRUE programmatically, so unless it's set in FDI
	files it never actually comes up)

Index: ChangeLog
===================================================================
RCS file: /cvs/hal/hal/ChangeLog,v
retrieving revision 1.280
retrieving revision 1.281
diff -u -d -r1.280 -r1.281
--- ChangeLog	2 Sep 2004 20:11:27 -0000	1.280
+++ ChangeLog	2 Sep 2004 21:45:49 -0000	1.281
@@ -1,5 +1,18 @@
 2004-09-02  Joe Shaw  <joeshaw at novell.com>
 
+	* hald/device.c (hal_device_property_get_bool): Aiee!  If the
+	property is not found, we want to return FALSE instead of -1.
+
+	* hald/property.c (hal_property_get_bool): Return FALSE if the
+	type isn't boolean in the assertion, not -1.
+
+	* hald/linux/common.c (rename_and_merge): Fix some broken logic
+	which only worked because it was depending on -1 being returned
+	from boolean functions.  If I understand its original intent
+	correctly, it should still work.  (Of course, info.not_available
+	is never set to TRUE programmatically, so unless it's set in FDI
+	files it never actually comes up)
+
 	* doc/TODO: Update the "Computer" HalDevice item, remove a 
 	duplicate item.
 




More information about the hal-commit mailing list