[FFmpeg-devel] [PATCH] lavfi/timecode: give a new alias "timecode_rate" to the the r/rate option

Clément Bœsch ubitux at gmail.com
Wed May 2 06:59:24 CEST 2012


On Wed, May 02, 2012 at 12:59:50AM +0200, Stefano Sabatini wrote:
> The option is related to the timecode, the new name clearly specifies the
> context. Also it allows to list the option close to the other timecode
> options.
> ---
>  doc/filters.texi          |    6 +++---
>  libavfilter/vf_drawtext.c |    1 +
>  2 files changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/doc/filters.texi b/doc/filters.texi
> index 2483fde..46be49e 100644
> --- a/doc/filters.texi
> +++ b/doc/filters.texi
> @@ -1329,9 +1329,6 @@ Default value is "render".
>  For more information consult the documentation for the FT_LOAD_*
>  libfreetype flags.
>  
> - at item rate, r
> -Set the timecode frame rate (timecode only).
> -
>  @item shadowcolor
>  The color to be used for drawing a shadow behind the drawn text.  It
>  can be a color name (e.g. "yellow") or a string in the 0xRRGGBB[AA]
> @@ -1352,6 +1349,9 @@ Set the initial timecode representation in "hh:mm:ss[:;.]ff"
>  format. It can be used with or without text parameter. @var{rate}

You can change the @var{rate} into @var{timecode_rate} here.

Otherwise LGTM.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120502/8923c4e6/attachment.asc>


More information about the ffmpeg-devel mailing list