[FFmpeg-devel] [PATCH] rl2 video fourcc for riff.c

compn tempn
Tue May 13 01:48:35 CEST 2008


On Mon, 12 May 2008 22:34:29 +0200, Aurelien Jacobs <aurel at gnuage.org> wrote:
>compn wrote:
>
>> On Mon, 12 May 2008 09:24:02 +0100, M?ns Rullg?rd <mans at mansr.com>
>> wrote:
>> >compn <tempn at twmi.rr.com> writes:
>> >
>> >> compn <tempn <at> twmi.rr.com> writes:
>> >>
>> >>> 
>> >>> this patch allows mplayer to play fate-suite/rl2/ with this
>> >>> codecs.conf entry:
>> >>
>> >> forgot patch:
>> >>
>> >> Index: libavformat/riff.c
>> >> ===================================================================
>> >> --- libavformat/riff.c  (revision 13118)
>> >> +++ libavformat/riff.c  (working copy)
>> >> @@ -160,6 +160,7 @@
>> >>      { CODEC_ID_VMNC,         MKTAG('V', 'M', 'n', 'c') },
>> >>      { CODEC_ID_TARGA,        MKTAG('t', 'g', 'a', ' ') },
>> >>      { CODEC_ID_CLJR,         MKTAG('c', 'l', 'j', 'r') },
>> >> +    { CODEC_ID_RL2,          MKTAG('r', 'l', '2', 'v') },
>> >>      { CODEC_ID_NONE,         0 }
>> >>  };
>> >
>> >That's not an AVI (or other RIFF-based) file, so this is the wrong
>> >place for that tag.  Do we really have to have this discussion
>> >*every* time?
>> 
>> oops, i knew there was a reason. maybe put a note in the source?
>> how about this patch for vp6a ?
>
>AFAIR, this is not a RIFF-based tag either.
>Or do you have a sample using it ?

no sample, i will give up now before i waste more of your time :(

-compn




More information about the ffmpeg-devel mailing list