[Bug 730758] tee: Handle ALLOCATION query

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sun Aug 27 17:57:36 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=730758

--- Comment #10 from Nicolas Dufresne (stormer) <nicolas at ndufresne.ca> ---
(In reply to Julien Isorce from comment #9)
> Review of attachment 357284 [details] [review]:
> 
> Should you remove "alloc-pad" property ? (seems to be mark unused anyway)

It seems that I forgot to attach it, but I have a patch that deprecate it
instead as removing it would be an API break.

> 
> Maybe add a unit test in gstreamer/tests/check/elements/tee.c ? At least one
> and then we can add more.

I'll do.

> 
> ::: plugins/elements/gsttee.c
> @@ +730,3 @@
> +      gst_allocation_params_init (&ctx.params);
> +
> +
> 
> Should it ignore not linked pads if allow-not-linked is true ?

I think this is difficult to implement since gst_pad_peer_query() returns a
boolean rather then GstFlowReturn. Though, GstPad return TRUE when it gets a
query on unlinked pads, which makes the code behave like if allow-not-linked
was set. I'm not sure if it should (I even doubt it should) behave differently
for the other case. I'm tempted to leave it this way.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list