[FFmpeg-cvslog] r13854 - trunk/libavcodec/opt.h

Diego Biurrun diego
Sun Jun 22 13:50:31 CEST 2008


On Sat, Jun 21, 2008 at 11:15:49AM +0200, stefano wrote:
> 
> Log:
> Extend documentation for AVOption.
> 
> --- trunk/libavcodec/opt.h	(original)
> +++ trunk/libavcodec/opt.h	Sat Jun 21 11:15:49 2008
> @@ -52,12 +52,20 @@ typedef struct AVOption {
> +
> +    /**
> +     * The offset relative to the context structure where the option
> +     * value is stored. It should be 0 for named constant.

constantS

Please review your patches/commits for plural 's' and third person
singular 's' mistakes, you skip them regularly.

> +    /**
> +     * the default value for non-constant and value for constant scalars
> +     */
>      double default_val;

I don't understand.  s/value for// ?

Diego




More information about the ffmpeg-cvslog mailing list