Re: libseed-list Digest, Vol 5, Issue 7

[Alan Forbes <alfraealba googlemail com>]

Works perfectly. 

Well done Alan!!

>  is a working version - it plays an mp3 quite nicely
> 
> http://www.akbkhome.com/svn/seed/seedbugs/gst_59537.js
> 
> The biggest change to the code is that the uri has to be in the format
> file:/tmp/test.mp3
> 
> 
> Note there are a few changes to seed and gir required (to handle the 
> messaging / error reporting)
> 
> These are:
> 
> a) seed_gi_argument_make_js - handle error types (and throw an excpetion) -
> Not sure if this is the best behaviour, as in the gst example, it get 
> called from the msg.parse_error() call..
> 
>  case GI_TYPE_TAG_ERROR:
>     seed_make_exception_from_gerror (ctx, exception, (GError*) 
> arg->v_pointer);
>     return false;
> 
> b) parse_error signature should look like this.
> * note that gerror and debug are both 'out' arguments.
> * the gchar** is defined as an array in my gir file - it's modified to a 
> standard type.
> 
> <method name="parse_error"
>               c:identifier="gst_message_parse_error"
>               doc="Extracts the GError and debug string from the 
> GstMessage. The values returned
> in the output arguments are copies; the caller must free them when done.
> MT safe.">
>         <return-value transfer-ownership="none">
>           <type name="none" c:type="void"/>
>         </return-value>
>         <parameters>
>           <parameter name="gerror"  direction="out" 
> transfer-ownership="full">
>             <type name="GLib.Error" c:type="GError**"/>
>           </parameter>
>           <parameter name="debug" direction="out"  
> transfer-ownership="full">
>               <type name="utf8" c:type="gchar**"/>
>           </parameter>
>         </parameters>
>       </method>
> 
> Regards
> Alan