[FFmpeg-devel] [PATCH] fix for transparencies in animated gifs

[Bjorn Roche]

Hey all,

I'm just reviewing this and noticing that the patch is stale and no longer
compiles. I am happy to update, but is there someone to review it if I do
so? I'd love to get this patch in. Thanks,

bjorn


On Thu, Dec 7, 2017 at 5:56 PM, Bjorn Roche <bjorn at giphy.com> wrote:

> Hey all,
>
> Just wondering what else is needed to get this merged? Is there anything
> else I can do?
>
> Thanks,
>
> bjorn
>
> On Thu, Nov 30, 2017 at 12:33 PM, Bjorn Roche <bjorn at giphy.com> wrote:
>
>> We've done a bit more testing on this and it looks good here.
>>
>> On Thu, Nov 30, 2017 at 12:26 PM, Bjorn Roche <bjorn at giphy.com> wrote:
>>
>>>
>>>
>>> On Mon, Nov 27, 2017 at 7:12 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
>>> wrote:
>>>
>>>> 2017-11-28 1:07 GMT+01:00 Bjorn Roche <bjorn at giphy.com>:
>>>> > On Mon, Nov 27, 2017 at 5:41 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com
>>>> >
>>>> > wrote:
>>>>
>>>> >> >> Does remuxing animated gif with transparency work with your patch?
>>>> >> >
>>>> >> > A command like this:
>>>> >> >
>>>> >> >     ffmpeg -i in.gif out.gif
>>>> >> >
>>>> >> > seems to produce the same output as before: transparency replaces
>>>> >> > with opacity.
>>>> >>
>>>> >> remuxing == -vcodec copy
>>>> >>
>>>> >> What about ffmpeg -i in.gif -vcodec copy out.gif ?
>>>> >>
>>>> >
>>>> > That doesn't work. Weather I use my patch or not it aborts:
>>>>
>>>> Of course, ticket #6640.
>>>
>>>
>>> I'm going to see if this is something I can tackle separately.
>>>
>>> bjorn
>>>
>>>
>>> --
>>>
>>>
>>> Bjorn Roche
>>>
>>> Sr. Video Pipeline Engineer
>>>
>>> bjorn at giphy.com
>>>
>>
>>
>>
>> --
>>
>>
>> Bjorn Roche
>>
>> Sr. Video Pipeline Engineer
>>
>> bjorn at giphy.com
>>
>
>
>
> --
>
>
> Bjorn Roche
>
> Sr. Video Pipeline Engineer
>
> bjorn at giphy.com
>



-- 


Bjorn Roche

Sr. Video Pipeline Engineer

bjorn at giphy.com