[FFmpeg-devel] [PATCH] AVI files with On2 headers

Benjamin Larsson banan
Tue Sep 25 17:27:35 CEST 2007


Aurelien Jacobs wrote:
> On Tue, 25 Sep 2007 11:20:02 +0100 (BST)
> M?ns Rullg?rd <mans at mansr.com> wrote:
>
>   
>> Aurelien Jacobs wrote:
>>     
>>> Hi,
>>>
>>> The attached patch add support for On2 variation of the AVI header.
>>>       
>> Hmm... where did I put my shotgun?
>>
>>     
>>> Sample to test it: http://samples.mplayerhq.hu/V-codecs/VP5/Cell-140.vp5
>>> Is this patch OK ?
>>>       
>> The patch looks reasonably clean, but I'm not AVI maintainer (thank
>> goodness).  That said, if there are more variants like this around,
>> I'd prefer a table-based solution.
>>     
>
> I tend to like such kind of tables, so here is a new version of the
> patch. It is more intrusive, but resulting code is nicer IMO.
> And it will simplify the incoming patch which adds AMV files support.
> So is this new patch OK ?
> If not, I will apply initial version of the patch.
>
> Aurel
>   

I like this better it's smaller then the first patch.

MvH
Benjamin Larsson

Patch missing exception.







More information about the ffmpeg-devel mailing list