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

[Vladimir Voroshilov]

2007/9/24, Aurelien Jacobs <aurel at gnuage.org>:
> On Mon, 24 Sep 2007 20:14:36 +0700
> "Vladimir Voroshilov" <voroshil at gmail.com> wrote:

[...]

> > amv_demux_ffmpeg.patch: AMV file demuxer.
>
> Better post one email per patch.
>
> > AMV files contains ugly headers: strh sections are filled by zero,
> > strf sections contains movie duration, image size and frame duration
> > for video and waveformatex structure for audio respectively.
> > Index is absent.
> > Patch uses simplified version of avi_read_header routine for
> > extracting needed info.
>
> I didn't looked carefully, but according to your description, this
> looks mostly like duplication of avi_read_header. This is not
> acceptable. You should modify avi_read_header directly instead.

Is attached version better ?

> > P.S. I also use new fourcc (AMVV) to allow further interaction with
> > mplayer.
>
> IIRC, this is not needed.

When i set codec_tag to MJPG, mjpeg decoder is loaded instead of amvv
(in MPlayer of course). codec_tag==AMVV plus apropriate section in codecs.conf
works fine.

-- 
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_demuxer_ffmpeg2.diff
Type: text/x-diff
Size: 3404 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070924/ef6913a8/attachment.diff>