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

Anuradha Suraparaju anuradha
Thu May 15 03:54:32 CEST 2008


Hi,

On Tue, 2008-05-13 at 01:55 +0200, Aurelien Jacobs wrote:
> 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).
> 

I am not sure if this is the right thread to post this question, but I
want to do something similar for the Dirac video codec, i.e. add a
fourcc tag to the codec_bmp_tags struct. The reason for this being, I'd
like to wrap Dirac in AVI, and also be able to play back Dirac (raw
streams and  wrapped in AVI) using mplayer (the mplayer patch is in
development). I'd like to use a fourcc code 'drac' for Dirac. Does this
need to be officially registered somewhere or can I just add it to the
list? 

> Aurel

Regards,
Anuradha





More information about the ffmpeg-devel mailing list