[FFmpeg-devel] [PATCH] Demuxer for AMV files

Vladimir Voroshilov voroshil
Sat Sep 29 17:39:44 CEST 2007


2007/9/29, Vitor Sessak <vitor1001 at gmail.com>:
> Hi
>
> Vladimir Voroshilov wrote:
> > Hi, All
> >
> > Attached patch implements demuxer for AMV files.
> > AMV fourcc tag moved to the end of AVI tags array as suggested by Aurelian.
> >
> > Discussion can be found
> > in "Video decoder and demuxer for AMV files" thread.
> >
> > If patch is ok please apply.
> >
> > P.S. AMV video decoder is required for this patch to be applied.
>

[...]

> >
> >              url_fskip(pb, 2 * 4);
> >              get_le32(pb);
> > +            get_le32(pb);
> > +            width=get_le32(pb);
> > +            height=get_le32(pb);
>
> Sorry for not bring this up before, but wouldn't it be better to call
> these variables "amv_width" and "amv_height" since they are AMV specific?
>

No problem. Here is patch.

-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amv_demux_fmpeg_as_avi5.diff
Type: text/x-diff
Size: 2643 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070929/b469d5d3/attachment.diff>



More information about the ffmpeg-devel mailing list