[FFmpeg-devel] [PATCH] Video decoder and demuxer for AMV files

Aurelien Jacobs aurel
Sat Sep 29 16:24:48 CEST 2007


On Sat, 29 Sep 2007 09:04:22 +0700
"Vladimir Voroshilov" <voroshil at gmail.com> wrote:

> 2007/9/29, Michael Niedermayer <michaelni at gmx.at>:
> > On Wed, Sep 26, 2007 at 11:01:52PM +0700, Vladimir Voroshilov wrote:
> > [...]
> >
> > > Index: mplayer/libavformat/avidec.c
> > > ===================================================================
> > > --- mplayer/libavformat/avidec.c      (revision 10592)
> > > +++ mplayer/libavformat/avidec.c      (working copy)
> > > @@ -58,6 +58,7 @@
> > >  } AVIContext;
> > >
> > >  static const char avi_headers[][8] = {
> > > +    { 'R', 'I', 'F', 'F',    'A', 'M', 'V', ' ' },
> > >      { 'R', 'I', 'F', 'F',    'A', 'V', 'I', ' ' },
> > >      { 'R', 'I', 'F', 'F',    'A', 'V', 'I', 'X' },
> > >      { 'R', 'I', 'F', 'F',    'A', 'V', 'I', 0x19},
> >
> > this hunk can be commited

This list is tested in order, and the check loop stop as soon
as it find a matching entry. So I think the most common headers
should be kept at the top of this list, and AMV should be at
the end of the list.

Aurel




More information about the ffmpeg-devel mailing list