[gst-devel] Re: [gst-cvs] company gst-plugins: gst-plugins/ gst-plugins/ext/mad/

in7y118 at public.uni-hamburg.de in7y118 at public.uni-hamburg.de
Thu Feb 5 09:37:35 CET 2004


Quoting "Ronald S. Bultje" <rbultje at ronald.bitfreak.net>:

> Hi,
> 
> On Wed, 2004-02-04 at 23:17, Benjamin Otte wrote:
> > -    return gst_pad_renegotiate (tag->srcpad) == GST_PAD_LINK_REFUSED;
> > +    return gst_pad_renegotiate (tag->srcpad) != GST_PAD_LINK_REFUSED;
> 
> Shouldn't you use GST_PAD_LINK_FAILED()/SUCCESSFUL() macros? DELAYED is
> "not ok" too, that's why these macro's exist...
> 
I think there was a reason for why I wanted to allow DELAYED. I'm not sure 
what it was though.

IMHO it would be much nicer if those functions returned TRUE/FALSE anyway for 
apps/plugins. Only the core needs to know about DELAYED and such.

Benjamin




More information about the gstreamer-devel mailing list