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

[Michael Niedermayer]

Hi

On Sun, Sep 30, 2007 at 12:30:34AM +0700, Vladimir Voroshilov wrote:
> 2007/9/30, Michael Niedermayer <michaelni at gmx.at>:
> > On Sun, Sep 30, 2007 at 12:03:52AM +0700, Vladimir Voroshilov wrote:
> > > 2007/9/29, Michael Niedermayer <michaelni at gmx.at>:
> > > > Hi
> > > >
> > > > On Sat, Sep 29, 2007 at 05:27:03PM +0200, Vitor Sessak wrote:
> > > > > Hi
> 
> [...]
> 
> > > > >
> > > > > 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?
> > > >
> > > > are they? what do they contain in normal avi files?
> > >
> > > They contans the same, but not used by lavf demuxer (it uses info from
> > > stream header).
> >
> > if they always contain width/height why should they be called
> > amv_width/height?
> >
> > avih_width / avih_height seem like better names to me ...
> >
> 
> Ok. Here it is.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070930/a117b944/attachment.pgp>