[FFmpeg-devel] [PATCH] lavfi: add internal function ff_fill_rgba_map()

Stefano Sabatini stefasab at gmail.com
Fri Oct 28 02:11:19 CEST 2011


On date Thursday 2011-10-27 00:53:08 +0200, Michael Niedermayer encoded:
> On Wed, Oct 26, 2011 at 08:04:14PM +0200, Stefano Sabatini wrote:
> > ---
> >  libavfilter/drawutils.c |   27 ++++++++++++++++-----------
> >  libavfilter/drawutils.h |    2 ++
> >  2 files changed, 18 insertions(+), 11 deletions(-)
> 
> nice idea, LGTM

Applied.
 
> independantly of this patch
> some documentation for the functions in drawutils.h would likely make
> them easier to use

The reason why I didn't document it is that I *want* the user to read
the implementation, indeed it is quite hackish (e.g. it only deals
with 32-bit ARGB formats, no support for RGB planar or 48-bits RGB
formats).
-- 
FFmpeg = Fierce & Funny Multipurpose Pure Experimenting Gangster


More information about the ffmpeg-devel mailing list