Re: [Bluez-devel] DBUS service support - Initial library

[Marcel Holtmann <marcel@holtmann.org>]

Hi Claudio,

> I am sending a new patch to be evaluated.
> Regarding glib dependency we need discuss this
> subject again. If you see the hcid code, there is a
> file (glib-ectomy.c) that implements the event
> processing, this code is a partial copy of glib.
> 
> What do you prefer? Repeat the glib-ectomy.c code
> or use glib?

the glib-ectomy is only still there, because I was too lazy to replace
it with a clean main loop using poll() implementation.

> For Maemo, glib is not problem. It is included in
> the SDK.

This is not a valid argument. Creating a massive dependency chain is
always bad. Using Glib inside bluez-utils is no option.

> Regarding pthread I put two versions to be evaluated:
> 1. using pthread
>    In my opinion because it save memory.
>    http://www.indt.org.br/maemo/bluez/bluez-dbus-thread.diff
>    
> 2. using fork
>    In my opinion this is not a feasible approach
>    because when listening is active two process 
>    will be running(a listening and another processing
>    D-Bus messages).
>    http://www.indt.org.br/maemo/bluez/bluez-dbus-fork.diff

Even without looking at the patches, I don't like any of it. I like to
see a nice main loop and then using poll(). We don't need threads or
forking out processes.

Regards

Marcel




-------------------------------------------------------
This SF.Net email is sponsored by Yahoo.
Introducing Yahoo! Search Developer Network - Create apps using Yahoo!
Search APIs Find out how you can build Yahoo! directly into your own
Applications - visit http://developer.yahoo.net/?fr=offad-ysdn-ostg-q22005
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel