hal: Branch 'master'

David Zeuthen david at kemper.freedesktop.org
Wed Apr 4 15:48:18 PDT 2007


 hald/hald.c |    3 ---
 1 files changed, 3 deletions(-)

New commits:
diff-tree 6538c9f78e2a3ef499c10923ecdee777d31e1fd3 (from 98454a5d597a3f96e3c683067ed9ca671b0b5b71)
Author: Danny Kukawka <danny.kukawka at web.de>
Date:   Wed Apr 4 18:48:10 2007 -0400

    fix problem with repeated property-changed signals
    
    > On Mittwoch, 4. April 2007, Richard Hughes wrote:
    > > With git I'm getting the following when I remove AC:
    > >
    > > 15:56:02.865: acpi_ACAD property ac_adapter.present = false
    > > 15:56:03.041: acpi_ACAD property ac_adapter.present = false
    > > 15:56:03.048: acpi_BAT1 property battery.charge_level.percentage = 96
    > > (0x60) 15:56:03.051: acpi_BAT1 property battery.charge_level.percentage =
    > > 96 (0x60)
    > >
    > > acpi_listen seems to think that there is one event from either device,
    > > so I'm unsure what the problem is here.
    >
    > I can confirm this, I take a look at the problem. It is IMO something in
    > the acpi code, could not see this atm for other devices.
    
    This is a general problem. The signal for device_property_changed is listed
    twice in hald.c in function hald_get_gdl().

diff --git a/hald/hald.c b/hald/hald.c
index 1a13a31..9135440 100644
--- a/hald/hald.c
+++ b/hald/hald.c
@@ -203,9 +203,6 @@ hald_get_gdl (void)
 				  "device_capability_added",
 				  G_CALLBACK (gdl_capability_added), NULL);
 		g_signal_connect (global_device_list,
-				  "device_property_changed",
-				  G_CALLBACK (gdl_property_changed), NULL);
-		g_signal_connect (global_device_list,
 				  "device_lock_acquired",
 				  G_CALLBACK (gdl_lock_acquired), NULL);
 		g_signal_connect (global_device_list,


More information about the hal-commit mailing list