[FFmpeg-devel] [PATCH] doc/filters: extend syntax description for transpose, and add examples

Stefano Sabatini stefasab at gmail.com
Sat Dec 15 21:01:34 CET 2012


On date Wednesday 2012-12-12 21:14:14 +0100, Clément Bœsch encoded:
> On Wed, Dec 12, 2012 at 12:37:07PM +0100, Stefano Sabatini wrote:
> > ---
> >  doc/filters.texi |   18 +++++++++++++++++-
> >  1 file changed, 17 insertions(+), 1 deletion(-)
> > 
> > diff --git a/doc/filters.texi b/doc/filters.texi
> > index b9b0ded..e4c16b6 100644
> > --- a/doc/filters.texi
> > +++ b/doc/filters.texi
> > @@ -3886,7 +3886,12 @@ Default mode is @code{merge}.
> >  
> >  Transpose rows with columns in the input video and optionally flip it.
> >  
> > -This filter accepts the following named parameters:
> > +The filter accepts parameters as a list of @var{key}=@var{value}
> > +pairs, separated by ":". If the key of the first options is omitted,
> > +the arguments are interpreted according to the following syntax:
> > + at example
> > +transpose=@var{dir}:@var{passthrough}
> > + at end example
> >  
> 
> example?

Fixed.
 
> Note: we should document the shorthand syntaxes in a consistent way in
> filters doc. Like making a section about shorthand, and referencing it
> with something like:
> 
>   This filter also supports the @ref{shorthand} syntax:
>   "@var{dir}:@var{passthrough}".
> 
> The shorthand section would give example with long, short, and mixed
> syntaxes.

Yes, work in this area is welcome.
 
> >  @table @option
> >  @item dir
> > @@ -3945,6 +3950,17 @@ Preserve landscape geometry (when @var{width} >= @var{height}).
> >  Default value is @code{none}.
> >  @end table
> >  
> > +For example to rotate by 90 degrees clockwise and preserve portrait
> > +layout:
> > + at example
> > +transpose=dir=1:passthrough=portrait
> > + at end example
> > +
> > +The command above can also be specified as:
> > + at example
> > +transpose=1:portrait
> > + at end example
> > +
> >  @section unsharp
> >  
> >  Sharpen or blur the input video.
> 
> LGTM assuming you fixed my top first comment.

Pushed, thanks for the review.
-- 
FFmpeg = Fantastic Funny Mastodontic Prodigious Elaborated Game


More information about the ffmpeg-devel mailing list