[Bug 751450] parse/grammar.y: Allow multiple links to happen.

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Jun 24 11:29:24 PDT 2015


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

--- Comment #5 from Jan Schmidt <thaytan at noraisin.net> ---
(In reply to Mathieu Duponchelle from comment #3)
> What token would you think about exactly ? "!!" maybe ?

Existing pipeline operators are all single char. It's possible to match a regex
instead, with a slightly bigger change to parse.l.

I'm not sure what single character might be a good one, so '!!' is probably
fine. Else, demux / multiqueue maybe?

> The thing is I filed that as enhancement, but it could nearly be considered
> a bug :)

Not really - it does one thing, you want it to do another. Not really a bug -
more a difference of opinion.

> If we chose to add a new token, I'd be concerned about its discoverability,
> but I understand your concern, even though it kind of reminds me of
> https://xkcd.com/1172/ :)

I'm more concerned about changing 10 year old behaviour causing even
harder-to-discover bugs.

(In reply to Thiago Sousa Santos from comment #4)
> What would happen if you had a capsfilter between the decodebin and
> encodebin? Would it detect that it should link all streams that match that
> caps?

Should do - a filtered link in the grammar just calls
gst_element_link_pads_filtered(), which will create a new capsfilter element
for each link.

-- 
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