[FFmpeg-devel] [PATCH] ogg muxer

[Baptiste Coudurier]

M?ns Rullg?rd wrote:
> Rich Felker <dalias at aerifal.cx> writes:
> 
> 
>>On Thu, Nov 01, 2007 at 03:27:05AM +0100, Michael Niedermayer wrote:
>>
>>>Hi
>>>
>>>On Thu, Nov 01, 2007 at 02:41:04AM +0100, Baptiste Coudurier wrote:
>>>[...]
>>>
>>>>>you also might want to add ogg regression tests after this gets commited
>>>>
>>>>vorbis encoding tests seems disabled atm, any plan to fix them ?
>>>
>>>well, no
>>>cant you just store something else in it? anything, it doesnt matter just
>>
>>Nope, ogg only supports a limited set of codecs with their own spec
>>for how to store them in ogg....
> 
> 
> Ogg can store FLAC, and we have a working FLAC encoder.
> 

Yes, here is another update with flac support. I tried using a switch,
but in fact it duplicates extradata and extrada_size check in all cases.

Regression tests included.

I put flac codec by default, I would prefer to put vorbis but ... ok for
everyone ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ogg_muxer3.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071110/4d62b2ba/attachment.asc>