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

Timothy Gu timothygu99 at gmail.com
Fri Jul 25 04:33:18 CEST 2014



On 07/24/2014 11:54 AM, Marton Balint wrote:
> 
> 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.

Consider this patch dropped, and do what you think is correct then.

[...]

Timothy


More information about the ffmpeg-devel mailing list