[FFmpeg-cvslog] r23983 - trunk/libavcodec/h264.c
Thu Jul 8 17:41:29 CEST 2010
On Thu, Jul 08, 2010 at 09:55:33AM -0400, Ronald S. Bultje wrote:
> On Fri, Jul 2, 2010 at 8:35 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Fri, Jul 02, 2010 at 01:54:34PM +0200, diego wrote:
> >> Author: diego
> >> Date: Fri Jul ?2 13:54:34 2010
> >> New Revision: 23983
> >> Log:
> >> Fix typo in Doxygen function parameter name.
> >> Modified:
> >> ? ?trunk/libavcodec/h264.c
> >> Modified: trunk/libavcodec/h264.c
> >> ==============================================================================
> >> --- trunk/libavcodec/h264.c ? Fri Jul ?2 13:53:00 2010 ? ? ? ?(r23982)
> >> +++ trunk/libavcodec/h264.c ? Fri Jul ?2 13:54:34 2010 ? ? ? ?(r23983)
> >> @@ -1428,7 +1428,7 @@ static int pred_weight_table(H264Context
> >> ?/**
> >> ? * Initialize implicit_weight table.
> >> - * @param field, 0/1 initialize the weight for interlaced MBAFF
> >> + * @param field ?0/1 initialize the weight for interlaced MBAFF
> >> ? * ? ? ? ? ? ? ? ?-1 initializes the rest
> >> ? */
> > Please refrain from making changes to code documentation without the
> > C developers checking them first.
> Some other comments might have been a little quick and should be
> reverted+discussed, but I think this change is correct. I'd leave this
> particular one in.
hmm, i thought that is dead obvious
the parameter field is described by
field, 0/1 initialize the weight for interlaced MBAFF -1 initializes the rest
0/1 initialize the weight for interlaced MBAFF -1 initializes the rest
that looks a bit odd in doxygen syntax
@param field field, 0/1 initialize the weight for interlaced MBAFF
but thats how it was meant imho
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-cvslog