[FFmpeg-cvslog] r23983 - trunk/libavcodec/h264.c

[Michael Niedermayer]

On Fri, Jul 09, 2010 at 09:03:44AM +0200, Diego Biurrun wrote:
> On Thu, Jul 08, 2010 at 09:05:42PM +0200, Reimar D?ffinger wrote:
> > On Thu, Jul 08, 2010 at 06:48:58PM +0100, M?ns Rullg?rd wrote:
> > > Michael Niedermayer <michaelni at gmx.at> writes:
> > > 
> > > > On Thu, Jul 08, 2010 at 06:30:58PM +0100, M?ns Rullg?rd wrote:
> > > >> Michael Niedermayer <michaelni at gmx.at> writes:
> > > >> 
> > > >> > lets assume the variable was named var0 instead of field
> > > >> > what was written there was:
> > > >> > @param field, 0/1 initialize the weight for interlaced MBAFF -1 initializes the rest
> > > >> >
> > > >> > what should have been written there (in that case) would have been:
> > > >> > @param var0 field, 0/1 initialize the weight for interlaced MBAFF -1 initializes the rest
> > > >> 
> > > >> Still takes me quite some effort to decipher.
> > > >
> > > > so how would you say it then?
> > > 
> > > I don't know since I still haven't figured out exactly what it's
> > > trying to say.
> > 
> > I guess it is
> > "indicates the field number (0 or 1) for interlaced MBAFF. -1 is used for the non-interlaced case."
> > Of course this is just a guess...
> 
> Michael, is that a better description?

no, its wrong
-1 is not limited to non interlaced, dont forget that in h264 there are
multiple forms of interlacing.

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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-cvslog/attachments/20100709/ea29ec3a/attachment.pgp>