[FFmpeg-devel] [PATCH 2/2] Extend color syntax, make it accept an alpha component specifier.

[Benoit Fouet]

Hi,

On Fri, 11 Jun 2010 00:34:21 +0200 Stefano Sabatini wrote:
> ---
>  libavfilter/parseutils.h |    8 ++++++-
> 
> diff --git a/libavfilter/parseutils.h b/libavfilter/parseutils.h
> index b5b494e..fc0c784 100644
> --- a/libavfilter/parseutils.h
> +++ b/libavfilter/parseutils.h
> @@ -46,7 +46,13 @@ char *av_get_token(const char **buf, const char *term);
>   * Puts the RGBA values that correspond to color_string in rgba_color.
>   *
>   * @param color_string a string specifying a color. It can be the name of
> - * a color (case insensitive match) or a 0xRRGGBB[AA] sequence.
> + * a color (case insensitive match) eventually followed by "@" and a
> + * string representing the alpha component, or a 0xRRGGBB[AA]
> + * sequence.

s/eventually/possibly/

> + * The alpha component may be a string composed by "0x" followed by an
> + * hexadecimal number or a base-10 number between 0 and 255, or a
> + * decimal number between 0.0 and 1.0, which represents the opacity
> + * value. If the alpha component is not specified then 255 is assumed.

maybe add a hint for user to know that 255 (or 0xff, or 1.0) is opaque.

Ben