[FFmpeg-devel] [misc-filters PATCH 4/8] Factorize out some code and implement the fill_line_with_color() function. Allow to share code with the pending color source.
Stefano Sabatini
stefano.sabatini-lala
Wed Jul 7 00:15:33 CEST 2010
On date Tuesday 2010-07-06 12:01:02 +0100, M?ns Rullg?rd encoded:
> Michael Niedermayer <michaelni at gmx.at> writes:
>
> > On Thu, Jul 01, 2010 at 10:50:08PM +0200, Stefano Sabatini wrote:
> >> ---
> >> libavfilter/vf_pad.c | 106 ++++++++++++++++++++++++++-----------------------
> >> 1 files changed, 56 insertions(+), 50 deletions(-)
> >
> > if you checked that this just moves code around then ok
>
> It also replaces RGB_TO_[YUV]_CCIR with plain RGB_TO_[YUV]. I assume
> this is a mistake.
I know how I messed it up, I copied code from the color source, which
was pretty old code which was written when the pad filter was not
still using _CCIR, I didn't noticed it when reading the patch (which I
did not less than five times), and the lavfitest was passing anyway.
Thanks for fixing, regards.
--
FFmpeg = Frenzy and Fabulous Multipurpose Picky Erroneous Governor
More information about the ffmpeg-devel
mailing list