[FFmpeg-devel] [PATCH] speex support for ogg demuxer

[Diego Biurrun]

On Tue, Feb 05, 2008 at 10:04:08PM +0000, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
> 
> > my (very sloppy) search did not reveal a specification for it, so here
> > is speex support for the demuxer based on the MPlayer code.
> 
> Spec is at http://speex.org/docs/manual/speex-manual/node8.html
> 
> > +ogg_codec_t speex_codec = {
> > +    .magic = "Speex",
> > +    .magicsize = 5,
> 
> Should be
> 
>     .magic = "Speex   ",
>     .magicsize = 8,
> 
> Note the 3 trailing spaces in the magic signature.
> 
> Patch OK apart from this.
> 
> Will you write a decoder now?

Yes, a Speex decoder is sorely missing :)

Diego