[MPlayer-dev-eng] [PATCH] Theora in Matroska

[Rich Felker]

On Wed, Sep 20, 2006 at 01:02:18PM +0400, Evgeniy Stepanov wrote:
> On Wednesday 20 September 2006 11:40, Nico Sabbi wrote:
> > Evgeniy Stepanov wrote:
> > >Hi,
> > >
> > >this patch adds Theora video support to demux_mkv.
> > >
> > >I just copypasted a piece code without thinking. Seems to work. Please
> > >review :)
> > >------------------------------------------------------------------------
> >
> > is there anything that works out of the box with matroska
> > without having to patch the demuxer?
> 
> Theora sure requires patching, because it's 3 first packets (bitstream 
> headers) are stored as codec private data. I don't see any mpeg-related hacks 
> in the demuxer, so that maybe what you are looking for.

AFAIK mkv already has the headers in the correct (single opaque block)
format. If our theora decoder doesn't understand them in this format
then it needs to be fixed..

Rich