[FFmpeg-devel] [PATCH 2/2] gif: use only one graphic control extension block per image.

[Clément Bœsch]

On Thu, Apr 18, 2013 at 11:20:19PM +0000, Paul B Mahol wrote:
> On 4/18/13, Clement Boesch <ubitux at gmail.com> wrote:
> > The encoder now doesn't produce any extra graphic control extension
> > block anymore. Only the image is encoded, and the muxer writing
> > its own GCE containing notably the timing information now includes the
> > optional palette transmitted through packet side data.
> >
> > This commit avoid setting clashes between the two GCE, and reduce the
> > size of the generated file with pal8 output.
> > ---
> >  libavcodec/gif.c  | 29 ++++++-----------------------
> >  libavformat/gif.c | 28 ++++++++++++++++++++++++++--
> >  2 files changed, 32 insertions(+), 25 deletions(-)
> >
> 
> lgtm

Pushed.

-- 
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/20130419/7b8d2d79/attachment.asc>