[FFmpeg-devel] [PATCH] lavfi: add drawutils

[Måns Rullgård]

Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:

> On date Sunday 2011-02-20 23:02:45 +0000, M?ns Rullg?rd encoded:
>> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
>> 
>> > Add drawutils.h and drawutils.c, and use them in the pad filter.
>> > The new functions are going to be shared by other filters.
>> > ---
>> >  libavfilter/Makefile    |    1 +
>> >  libavfilter/drawutils.c |  117 +++++++++++++++++++++++++++++++++++++++++++
>> >  libavfilter/drawutils.h |   43 ++++++++++++++++
>> >  libavfilter/vf_pad.c    |  126 ++++++++---------------------------------------
>> >  4 files changed, 182 insertions(+), 105 deletions(-)
>> >  create mode 100644 libavfilter/drawutils.c
>> >  create mode 100644 libavfilter/drawutils.h
>> 
>> Is this just moving some code from vf_pad.c to drawutils.c?
>
> Yes, but I also changed the signature of fill_line_with_color():
>
> ff_fill_line_with_color(uint8_t *line[4], int pixel_step[4], int w,
>                         uint8_t dst_color[4],
>                         enum PixelFormat pix_fmt, uint8_t rgba_color[4],
>                         int *is_packed_rgba, uint8_t rgba_map[4]);
>
> by adding rgba_map (which is useful for the drawtext filter).

If you say so, I'm fine with it.

-- 
M?ns Rullg?rd
mans at mansr.com