[PATCH][RFC]Use a hash table for device properties.

David Zeuthen david at fubar.dk
Wed Aug 31 08:35:08 PDT 2005


On Wed, 2005-08-31 at 10:22 +0200, Cornelia Huck wrote:
> 2005/8/30, David Zeuthen <david at fubar.dk>:
> 
> > Maybe we don't need the hal_hash_property_free function if it's empty?
> 
> We need a function that simply returns "TRUE" so that the property gets
> removed.

OK, good point :-)

> > So.. aren't we leaking the property name (hashtable key) here?
> 
> I don't think so. hal_property_free() already should take care of that.

OK

> 
> > We should probably avoid using C99 constructs in the shared code (ok in
> > hald/linux2 though)..
> 
> OK, will rewrite those.
> 
> Thanks for looking through the patch!

Thanks,
David




More information about the hal mailing list