[FFmpeg-devel] [PATCH] lavfi/split: fix minor documentation issues

Stefano Sabatini stefasab at gmail.com
Mon Apr 15 23:44:44 CEST 2013


On date Monday 2013-04-15 22:36:47 +0200, Clément Bœsch encoded:
> On Mon, Apr 15, 2013 at 09:31:14PM +0200, Stefano Sabatini wrote:
> > Use verbal form to describe "outputs" options, and mention it in the
> > manual.
> > ---
> >  doc/filters.texi    |    4 ++--
> >  libavfilter/split.c |    2 +-
> >  2 files changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/doc/filters.texi b/doc/filters.texi
> > index 9f7e22b..8b6546a 100644
> > --- a/doc/filters.texi
> > +++ b/doc/filters.texi
> > @@ -5721,8 +5721,8 @@ subtitles=filename=sub.srt
> >  
> >  Split input video into several identical outputs.
> >  
> > -The filter accepts a single parameter which specifies the number of outputs. If
> > -unspecified, it defaults to 2.
> > +The filter accepts a single option @option{outputs} which specifies
> > +the number of outputs. If unspecified, it defaults to 2.
> >  
> >  For example
> >  @example
> > diff --git a/libavfilter/split.c b/libavfilter/split.c
> > index 7268ec1..6a52073 100644
> > --- a/libavfilter/split.c
> > +++ b/libavfilter/split.c
> > @@ -93,7 +93,7 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *frame)
> >  #define OFFSET(x) offsetof(SplitContext, x)
> >  #define FLAGS AV_OPT_FLAG_AUDIO_PARAM | AV_OPT_FLAG_VIDEO_PARAM
> >  static const AVOption options[] = {
> > -    { "outputs", "Number of outputs", OFFSET(nb_outputs), AV_OPT_TYPE_INT, { .i64 = 2 }, 1, INT_MAX, FLAGS },
> > +    { "outputs", "set number of outputs", OFFSET(nb_outputs), AV_OPT_TYPE_INT, { .i64 = 2 }, 1, INT_MAX, FLAGS },
> >      { NULL },
> >  };
> >  
> 
> OK.

Applied, thanks.
-- 
FFmpeg = Funny and Free Mere Peaceful Erotic Guru


More information about the ffmpeg-devel mailing list