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

[Vladimir Voroshilov]

2007/9/25, Aurelien Jacobs <aurel at gnuage.org>:
> Vitor Sessak wrote:
>
> > Hi
> >
> > Aurelien Jacobs wrote:
> > > Vitor Sessak wrote:
> > >
> > >> Hi
> > >>
> > >> Vladimir Voroshilov wrote:
> > >>> Hi, All.
> > >>>
> > >>> amv_codec_ffmpeg.patch: Decoder for modified MJPEG, used it AMV files.
> > >> [...]
> > >>
> > >> Interestingly, while you were working with the video patch, I had a look
> > >> at the audio. See attached patch.
> > >
> > > According to Vladimir's patch, audio is nothing more than
> > > CODEC_ID_ADPCM_IMA_WS (but I didn't checked it).
> >
> > I got the audio distorted with his patch. Either I did something wrong
> > in applying his patch or he focused in video first... Did you try it
> > with the samples.mplayer.hu files?
>
> No, as I've said, I didn't checked. I've only read the patch.
> If that's true, then Vladimir should use CODEC_ID_ADPCM_IMA_AMV
> in his patch.

Yes. Vitor is right.
I didn't pay much attention to audio part (just got first working codec).
Vitor's patch gives much better sound.

-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at gmail.com, voroshil at jabber.ru
ICQ: 95587719