diff mbox

[FFmpeg-devel] doc/codecs: Change common boolean parameters listed to "bool"

Message ID 20170510010846.31065-1-michael@niedermayer.cc
State Accepted
Headers show

Commit Message

Michael Niedermayer May 10, 2017, 1:08 a.m. UTC
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
---
 doc/codecs.texi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

James Almer May 10, 2017, 1:13 a.m. UTC | #1
On 5/9/2017 10:08 PM, Michael Niedermayer wrote:
> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> ---
>  doc/codecs.texi | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/doc/codecs.texi b/doc/codecs.texi
> index 1a838e57c3..ec49b82227 100644
> --- a/doc/codecs.texi
> +++ b/doc/codecs.texi
> @@ -1258,7 +1258,7 @@ Interlaced video, top coded first, bottom displayed first
>  Interlaced video, bottom coded first, top displayed first
>  @end table
>  
> -@item skip_alpha @var{integer} (@emph{decoding,video})
> +@item skip_alpha @var{bool} (@emph{decoding,video})
>  Set to 1 to disable processing alpha (transparency). This works like the
>  @samp{gray} flag in the @option{flags} option which skips chroma information
>  instead of alpha. Default is 0.
> @@ -1279,7 +1279,7 @@ ffprobe -dump_separator "
>  Maximum number of pixels per image. This value can be used to avoid out of
>  memory failures due to large images.
>  
> -@item apply_cropping @var{integer} (@emph{decoding,video})
> +@item apply_cropping @var{bool} (@emph{decoding,video})
>  Enable cropping if cropping parameters are a multiply of the required
>  alignment for the left and top parameters. If the alignment is not met the
>  cropping will be partially applied to maintain alignment.
> 

LGTM.
Michael Niedermayer May 10, 2017, 1:25 a.m. UTC | #2
On Tue, May 09, 2017 at 10:13:20PM -0300, James Almer wrote:
> On 5/9/2017 10:08 PM, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
> > ---
> >  doc/codecs.texi | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/doc/codecs.texi b/doc/codecs.texi
> > index 1a838e57c3..ec49b82227 100644
> > --- a/doc/codecs.texi
> > +++ b/doc/codecs.texi
> > @@ -1258,7 +1258,7 @@ Interlaced video, top coded first, bottom displayed first
> >  Interlaced video, bottom coded first, top displayed first
> >  @end table
> >  
> > -@item skip_alpha @var{integer} (@emph{decoding,video})
> > +@item skip_alpha @var{bool} (@emph{decoding,video})
> >  Set to 1 to disable processing alpha (transparency). This works like the
> >  @samp{gray} flag in the @option{flags} option which skips chroma information
> >  instead of alpha. Default is 0.
> > @@ -1279,7 +1279,7 @@ ffprobe -dump_separator "
> >  Maximum number of pixels per image. This value can be used to avoid out of
> >  memory failures due to large images.
> >  
> > -@item apply_cropping @var{integer} (@emph{decoding,video})
> > +@item apply_cropping @var{bool} (@emph{decoding,video})
> >  Enable cropping if cropping parameters are a multiply of the required
> >  alignment for the left and top parameters. If the alignment is not met the
> >  cropping will be partially applied to maintain alignment.
> > 
> 
> LGTM.

applied

thx

[...]
diff mbox

Patch

diff --git a/doc/codecs.texi b/doc/codecs.texi
index 1a838e57c3..ec49b82227 100644
--- a/doc/codecs.texi
+++ b/doc/codecs.texi
@@ -1258,7 +1258,7 @@  Interlaced video, top coded first, bottom displayed first
 Interlaced video, bottom coded first, top displayed first
 @end table
 
-@item skip_alpha @var{integer} (@emph{decoding,video})
+@item skip_alpha @var{bool} (@emph{decoding,video})
 Set to 1 to disable processing alpha (transparency). This works like the
 @samp{gray} flag in the @option{flags} option which skips chroma information
 instead of alpha. Default is 0.
@@ -1279,7 +1279,7 @@  ffprobe -dump_separator "
 Maximum number of pixels per image. This value can be used to avoid out of
 memory failures due to large images.
 
-@item apply_cropping @var{integer} (@emph{decoding,video})
+@item apply_cropping @var{bool} (@emph{decoding,video})
 Enable cropping if cropping parameters are a multiply of the required
 alignment for the left and top parameters. If the alignment is not met the
 cropping will be partially applied to maintain alignment.