[FFmpeg-devel] [PATCH] doc/codecs: Updates for skip_alpha

[Stefano Sabatini]

On date Wednesday 2013-10-23 20:42:52 -0700, Timothy Gu encoded:
> On Sun, Oct 20, 2013 at 12:16 PM, Stefano Sabatini <stefasab at gmail.com> wrote:
> > On date Friday 2013-10-18 18:00:35 -0700, Timothy Gu encoded:
> >> Merged-by: Michael Niedermayer <michaelni at gmx.at>
> >> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> >> Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
> >> ---
> >>  doc/codecs.texi | 4 +++-
> >>  1 file changed, 3 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/doc/codecs.texi b/doc/codecs.texi
> >> index 74dd775..61dd46a 100644
> >> --- a/doc/codecs.texi
> >> +++ b/doc/codecs.texi
> >> @@ -1076,7 +1076,9 @@ Interlaced video, bottom coded first, top displayed first
> >>  @end table
> >>
> >>  @item skip_alpha @var{integer} (@emph{decoding,video})
> >> -causes alpha plane not to be decoded, like "gray" which skips chroma.
> >
> >> +Set to 1 to disable processing alpha (transparency). This works like
> >
> > Probably "decoding" is more specific in this case.
> 
> Left as is per Reimar's comment
> 
> >
> >> + at samp{gray} flag in @option{flags} option which skips chroma information
> >
> > the @samp{gray} flag in the @option{flags} option?
> 
> Fixed.
> 
> >
> >> +instead of alpha in this case. Default is 0.
> >>  @end table
> >
> > LGTM otherwise.
> 
> Timothy

> From 82d2aee423a44a826d8224993493957b30477cf0 Mon Sep 17 00:00:00 2001
> From: Timothy Gu <timothygu99 at gmail.com>
> Date: Fri, 18 Oct 2013 17:56:14 -0700
> Subject: [PATCH] doc/codecs: Updates for skip_alpha
> 
> Merged-by: Michael Niedermayer <michaelni at gmx.at>
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
> ---
>  doc/codecs.texi | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/codecs.texi b/doc/codecs.texi
> index 74dd775..e79e462 100644
> --- a/doc/codecs.texi
> +++ b/doc/codecs.texi
> @@ -1076,7 +1076,9 @@ Interlaced video, bottom coded first, top displayed first
>  @end table
>  
>  @item skip_alpha @var{integer} (@emph{decoding,video})
> -causes alpha plane not to be decoded, like "gray" which skips chroma.
> +Set to 1 to disable processing alpha (transparency). This works like the
> + at samp{gray} flag in the @option{flags} option which skips chroma information
> +instead of alpha. Default is 0.
>  @end table

LGTM, will push it soon.
-- 
FFmpeg = Free and Foolish Mind-dumbing Ponderous Ecstatic Geek