GNOME Bugzilla – Bug 705565
matroskademux: add missing parsed field in mpeg audio caps
Last modified: 2013-08-12 09:44:44 UTC
Created attachment 250968 [details] [review] add missing parsed field in mpeg audio caps Pretty straightforward. This change is needed since the caps subset check were tightened to avoid plugging in a mpegaudioparse after matroskademux.
There's nothing in the Matroska spec that requires MPEG audio frames to be properly framed. You could have partial frames in a Matroska EBML element. Also decodebin will *always* plug parsers now, even if you set parsed=true on the caps.
(In reply to comment #1) > There's nothing in the Matroska spec that requires MPEG audio frames to be > properly framed. You could have partial frames in a Matroska EBML element. > Ok, I wasn't sure about this. > Also decodebin will *always* plug parsers now, even if you set parsed=true on > the caps. Not when the audio sink can also decode, which is my case
(In reply to comment #2) > > Also decodebin will *always* plug parsers now, even if you set parsed=true on > > the caps. > > Not when the audio sink can also decode, which is my case Well, if your audiosink accepts unparsed caps, decodebin won't plug a parser. That's true :)