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

Stefano Sabatini stefasab at gmail.com
Thu May 3 10:49:29 CEST 2012


On date Wednesday 2012-05-02 06:59:24 +0200, Clément Bœsch encoded:
> 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.

Fixed and pushed.
-- 
FFmpeg = Fiendish Fiendish Mastering Pure Erudite Glue


More information about the ffmpeg-devel mailing list