[FFmpeg-cvslog] r25241 - in trunk: cmdutils.c cmdutils.h ffplay.c

Diego Biurrun diego
Fri Oct 1 20:59:46 CEST 2010


On Tue, Sep 28, 2010 at 03:16:31PM -0400, Alexander Strange wrote:
> 
> On Sep 28, 2010, at 3:05 PM, Diego Biurrun wrote:
> 
> > On Tue, Sep 28, 2010 at 04:05:12AM +0200, astrange wrote:
> >> 
> >> Log:
> >> Extract timestamp correction code from ffplay.c to cmdutils.c
> >> 
> >> --- trunk/cmdutils.h	Tue Sep 28 02:33:21 2010	(r25240)
> >> +++ trunk/cmdutils.h	Tue Sep 28 04:05:12 2010	(r25241)
> >> @@ -226,4 +226,28 @@ int read_yesno(void);
> >> +
> >> +/**
> >> + * Resets the state of the PtsCorrectionContext.
> >> + */
> >> +void init_pts_correction(PtsCorrectionContext *ctx);
> > 
> > ctx is left undocumented.
> 
> I never liked the idea of "@param ctx the context", it hardly adds anything to the documentation.
> But I can add it if you want.

It avoids a Doxygen warning.  Some day, if our hearts stay pure, we
might get the number of Doxygen warnings under control and they
might actually become useful.

Diego



More information about the ffmpeg-cvslog mailing list