[FFmpeg-devel] [PATCH] ACDV (JPEG) codec

[compn]

On Tue, 09 Dec 2008 23:51:04 -0800, Mike Melanson wrote:
>Diego Biurrun wrote:
>> On Tue, Dec 09, 2008 at 09:26:21PM -0800, Mike Melanson wrote:
>>> One-line patch to riff.c to support ACDV codec which is actually just  
>>> full JPEG files stored in individual AVI frames.

ahhh, forgot to test this one against jpeg ;\

>> Why send a patch for this triviality?  Just commit...
>
>A few reasons:
>
>1) I'm never quite sure what counts as triviality.
>
>2) For all I know, I could have messed up the order in riff.c without 
>recognizing it.

one of the rare files in ffmpeg without an alphabetical or
backward-compatable list.

>3) This gives you a chance to jump on me about forgetting to document 
>something (although I checked around and it doesn't look like we 
>document FourCCs that map to known codecs, though perhaps we should).

i thought the idea was to map codecs that ffmpeg has a decoder for.
but then there is codec_id_indeo5 so i have no idea.

-compn