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

Måns Rullgård mans
Thu Jul 8 17:44:31 CEST 2010


Michael Niedermayer <michaelni at gmx.at> writes:

> On Thu, Jul 08, 2010 at 09:55:33AM -0400, Ronald S. Bultje wrote:
>> Hi,
>> 
>> 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
> not
> 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

That doesn't make sense to me.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-cvslog mailing list