[FFmpeg-devel] [PATCH] Add drawtext filter from the libavfilter soc repo.

[Stefano Sabatini]

On date Tuesday 2011-03-08 12:08:57 -0500, Justin Ruggles encoded:
> Hi,
[...]
> > + at item fontcolor
> > +The color to be used for drawing fonts.
> > +Either a string (e.g. "red") or in 0xRRGGBB[AA] format
> > +(e.g. "0xff000033"), possibly followed by an alpha specifier.
> > +The default value of @var{fontcolor} is "black".
> 
> Might be simpler to add a section regarding the color format string in
> the docs somewhere, then link to it from the various parameters in
> libavfilter that specify a color rather than having to explain the same
> thing for each parameter.
> 
> Also, that example uses alpha, so what kind of "alpha specifier" would
> follow it?

A possible idea: -colors in cmdutils.[hc], then a section color.texi,
which looks a bit overkill, or alternatively a misc.texi. I'd rather
avoid to list all the colors in the manual to avoid to make it even
longer.

For the other doxy changes, I'd prefer to keep them in a separate
patch for easing integration with the videolan repo.
-- 
FFmpeg = Faithless and Fantastic Monstrous Perennial Exciting Gadget