[Bug 793333] matroskademux: Allow Matroska headers to be read more than once

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sat Sep 22 12:16:47 UTC 2018


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

--- Comment #28 from Thibault Saunier <tsaunier at gnome.org> ---
(In reply to Tim-Philipp Müller from comment #26)
> I'm a bit concerned about this commit and wonder if it needs more discussion:
> 
>   matroskademux: Parse successive Tracks elements
> 
>   This patch allows matroskademux to parse a second Tracks element,
>   erroring out if the tracks are not compatible (different number, type or
>   codec) and emitting new caps and tag events should they have changed.
> 
> So now we push potentially different caps (even if same codec) on an
> existing source pad, right?
> 
> Is that something we do anywhere else?
> 
> My understanding is that we have explicitly not been doing this because the
> old decodebin couldn't handle this.

I had checked and we do this in qtdemux... if the parent is stream aware, but I
missed that second part reading the code. I guess we should add that check.

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