[FFmpeg-devel] [PATCH] ffplay: remove dead code

Marton Balint cus at passwd.hu
Thu Jul 24 20:54:08 CEST 2014


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 <timothygu99 at 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


More information about the ffmpeg-devel mailing list