Commit 3e5b02bd authored by Jean Delvare's avatar Jean Delvare Committed by Michael Niedermayer

avfilter/delogo: Fix show option when band is small

The code assumed that the outermost interpolated pixels were always in
the fuzzy area defined by the band option. However if the band value
is small, there may be no fuzzy area on a given plane. In that case,
option show did not work, no rectangle was drawn (or only on the luma
plane, depending on the band value and chroma plane subsampling
factors.)

Fix the problem by not making any assumption on where the outermost
interpolated pixels will be.

The new code was verified to produce the same result as the original
code when the band value is not small.
Signed-off-by: 's avatarJean Delvare <jdelvare@suse.de>
Signed-off-by: 's avatarMichael Niedermayer <michael@niedermayer.cc>
parent 578f721f
/* /*
* Copyright (c) 2002 Jindrich Makovicka <makovick@gmail.com> * Copyright (c) 2002 Jindrich Makovicka <makovick@gmail.com>
* Copyright (c) 2011 Stefano Sabatini * Copyright (c) 2011 Stefano Sabatini
* Copyright (c) 2013 Jean Delvare <jdelvare@suse.com> * Copyright (c) 2013, 2015 Jean Delvare <jdelvare@suse.com>
* *
* This file is part of FFmpeg. * This file is part of FFmpeg.
* *
...@@ -101,6 +101,12 @@ static void apply_delogo(uint8_t *dst, int dst_linesize, ...@@ -101,6 +101,12 @@ static void apply_delogo(uint8_t *dst, int dst_linesize,
xdst = dst+logo_x1+1, xdst = dst+logo_x1+1,
xsrc = src+logo_x1+1; x < logo_x2-1; x++, xdst++, xsrc++) { xsrc = src+logo_x1+1; x < logo_x2-1; x++, xdst++, xsrc++) {
if (show && (y == logo_y+1 || y == logo_y+logo_h-2 ||
x == logo_x+1 || x == logo_x+logo_w-2)) {
*xdst = 0;
continue;
}
/* Weighted interpolation based on relative distances, taking SAR into account */ /* Weighted interpolation based on relative distances, taking SAR into account */
weightl = (uint64_t) (logo_x2-1-x) * (y-logo_y1) * (logo_y2-1-y) * sar.den; weightl = (uint64_t) (logo_x2-1-x) * (y-logo_y1) * (logo_y2-1-y) * sar.den;
weightr = (uint64_t)(x-logo_x1) * (y-logo_y1) * (logo_y2-1-y) * sar.den; weightr = (uint64_t)(x-logo_x1) * (y-logo_y1) * (logo_y2-1-y) * sar.den;
...@@ -138,8 +144,6 @@ static void apply_delogo(uint8_t *dst, int dst_linesize, ...@@ -138,8 +144,6 @@ static void apply_delogo(uint8_t *dst, int dst_linesize,
dist = FFMAX(dist, y-(logo_y+logo_h-1-band)); dist = FFMAX(dist, y-(logo_y+logo_h-1-band));
*xdst = (*xsrc*dist + interp*(band-dist))/band; *xdst = (*xsrc*dist + interp*(band-dist))/band;
if (show && (dist == band-1))
*xdst = 0;
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment