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

[Diego Biurrun]

On Fri, Jul 02, 2010 at 02:35:24PM +0200, Michael Niedermayer wrote:
> On Fri, Jul 02, 2010 at 01:54:34PM +0200, diego wrote:
> > 
> > Log:
> > Fix typo in Doxygen function parameter name.
> > 
> > --- 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.

I cannot see a mistake here, nor can anybody else:

15:53 <@DonDiego> btw, can anybody see the mistake in r23983? afaict it's correct
15:54 <@mru> removing the comma looks correct
15:54 <@benoit-> FWIW, to me too.
15:56 <@BBB> the commit is fine, the comma is wrong

Diego