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

Ronald S. Bultje rsbultje
Thu Jul 8 15:55:33 CEST 2010


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.

Thanks,
Ronald



More information about the ffmpeg-cvslog mailing list