Re: [gupnp] Introspection is here!

From: ذیشان ءلی (Zeeshan Ali) <zeenix@gstreamer.net>
Date: Sat Jul 07 2007 - 07:52:19 EDT

Hello!

On 7/7/07, Jorn Baayen <jorn@openedhand.com> wrote:
>
> Cool! Cleaned up a bit (including reformatting to fit 80 chars ;) ),
> changed the code to use the Context SoupSession instead of creating new
> sessions, and added code for conversion of strings to GUPnP types.

  Cool!

> Also I'm not
> quite happy with the way errors are ignored when getting introspection
> objects in ServiceInfo. I'm thinking we should add GError arguments to
> the introspection retrieval methods, and replace the signal with a
> callback.

   I wasn't happy with the way errors are handled either and always
thought i'll talk to you about using GError in my next mail to you but
always forgot. :) But it's not just the introspection object that
needs prorper error handling, somehow emiting a g_warning doesn't seem
the right way to me. Most of our API works asynchronously and that
doesn't make it easy to report errors to the app. Gst guys have solved
this problem through their message bus, but what method can we use?

   That signal didn't let me sleep at night either so i was thinking
of replacing it too. :)

-- 
Regards,
Zeeshan Ali
--
To unsubscribe send a mail to gupnp+unsubscribe\@o-hand.com
Received on Sat, 7 Jul 2007 14:52:19 +0300

This archive was generated by hypermail 2.1.8 : Sat Jul 07 2007 - 09:00:06 EDT