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

[Aurelien Jacobs]

compn wrote:

> 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 :(

BTW: if your goal is simply to add support for those fourcc-less
codecs in MPlayer, then I think libmpdemux/mp_taglists.c is what
you're looking for (in MPlayer tree).

Aurel