[PATCH] Re: Repeated property-changed signals

[Danny Kukawka]

On Mittwoch, 4. April 2007, Danny Kukawka wrote:
> 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().

See attached patch.

Danny
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-remove_double_device_property_changed.diff
Type: text/x-diff
Size: 511 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20070404/4a875be6/hal-remove_double_device_property_changed.bin