[FFmpeg-devel] [PATCH] Cleanse GIF muxer and encoder.

Clément Bœsch ubitux at gmail.com
Thu Apr 18 00:28:02 CEST 2013

On Wed, Apr 17, 2013 at 10:06:07AM +0000, Paul B Mahol wrote:
> On 4/17/13, Clement Boesch <ubitux at gmail.com> wrote:
> > This commit removes the badly duplicated code between the encoder and
> > the muxer. That may sound surprising, but the encoder is now responsible
> > from the encoding of the picture when muxing to a .gif file. It also
> > does not require anymore a manual user intervention such as a -pix_fmt
> > rgb24 to work properly. To summarize, output gif are now easier to
> > generate, code is saner and simpler, and files are smaller (thanks to
> > the lzw encoding which was unused so far with the default .gif output).
> > We can certainly make things even better, but this is the first step.
> >
> > FATE is updated because of the output being produced by the encoder and
> > not the muxer (no lzw in the muxer), and in the seek test only the size
> > mismatches.
> >
> > ---
> >
> If you plan on improving gif encoder (to use disposals, etc, there is
> bunch of code and examples floating around) than LGTM.

Yes I do plan to work on disposals, but there are all kind of other
potential improvements for GIF in FFmpeg...

Thanks, applied.

Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130418/c556af7d/attachment.asc>

More information about the ffmpeg-devel mailing list