On Thu, 24 Jul 2014, Michael Niedermayer wrote:
Hi On Thu, Jul 17, 2014 at 06:02:02PM -0700, Timothy Gu wrote:Signed-off-by: Timothy Gu <timothyg...@gmail.com> --- ffplay.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index af0e199..8abbeae 100644 --- a/ffplay.c +++ b/ffplay.c @@ -775,8 +775,6 @@ static void blend_subrect(AVPicture *dst, const AVSubtitleRect *rect, int imgw, lum[0] = ALPHA_BLEND(a, lum[0], y, 0); YUVA_IN(y, u, v, a, p + BPP, pal); - u1 += u; - v1 += v; a1 += a; lum[1] = ALPHA_BLEND(a, lum[1], y, 0); cb[0] = ALPHA_BLEND(a1 >> 2, cb[0], u, 1);
I have a feeling that the ALPHA_BLENDs should use u1 and v1 respectively, and not u and v, so this is not the proper fix.
CC-ing ffplay maintainer, so he doesnt miss this
yeah, sorry for the delay. Regards, Marton _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
