[FFmpeg-devel] [PATCH] add RV10 fourcc to libavformat/riff.c

Corey Hickey bugfood-ml
Sun May 13 23:01:29 CEST 2007

Michael Niedermayer wrote:
>> RV10 is missing. Is this the right place to put it?
> hmm, may i ask why that should be there?

I don't know at all. That's why I asked. :) It was just as easy to send
a patch as well.

> iam not against adding it if there are some rv10 in avi around
> iam also not against adding it if stream copy of rv10 into avi
> would then produce a playable avi file

Apparently rv10 in avi works, but it does so with or without my patch.

> OTOH if this is because mplayer needs it with lavf (de) muxer then
> this is not correct ...

I can put it in demux_lavf.c, codec_bmp_tags[] if that's a better place.
 Either place works for mplayer (including the bug Compn pointed out).


More information about the ffmpeg-devel mailing list