[Ffmpeg-devel] [PATCH] GIF encoder

[Baptiste Coudurier]

Michael Niedermayer wrote:
> Hi
> 
> On Wed, Nov 01, 2006 at 09:11:27PM +0100, Baptiste Coudurier wrote:
>> Hi
>>
>> GIF encoder ported from lavf to lavc.
>> Very simple encoder (no compression), if someone is motivated to code
>> LZW compression it would be nice.
>>
>> Now we can remove AVImageFormat from lavf without regression.
> 
> what about animated gifs, do they work too with your encoder?
> 
> and do the regression tests pass if the new encoder is used instead
> of the old? if not why not?

In fact, goal is to remove AVImageFormat, old gif muxer/demuxer in lavf
are still present and they provide animated gif support.
Now with encoder, image format can be dropped in favor of image2.

> if yes to both then apply it
> 
> [...]
>> -    put_byte(pb, 0x00); /* end of image block */
>> -
>> +    bytestream_put_byte(bytestream, 0x00); /* end of image block */
>> +    bytestream_put_byte(bytestream, 0x3b);
>>      return 0;
>>  }
> 
> this looks different?
> 

In fact line was moved from gif_write to gif_image_write_image, but I
can let it in encode_frame if you want.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312