[FFmpeg-devel] [PATCH] avcodec[/format]/webpenc: use WebPAnimEncoder API to generate animated WebP

[Urvang Joshi]

Hi,
Here's an updated patch that used 'HAVE_WEBP_MUX_H' as you suggested. I
also updated the patch to work with the latest libwebp code (from
https://gerrit.chromium.org/gerrit/gitweb?p=webm/libwebp.git)

Thanks,
Urvang

On Tue, Mar 31, 2015 at 1:08 PM Pascal Massimino <pascal.massimino at gmail.com>
wrote:

> Hi,
>
> On Sat, Mar 28, 2015 at 4:51 AM, James Almer <jamrial at gmail.com> wrote:
>
> > On 27/03/15 7:02 AM, Pascal Massimino wrote:
> > > diff --git a/libavformat/webpenc.c b/libavformat/webpenc.c
> > > index ee110de..d4411a9 100644
> > > --- a/libavformat/webpenc.c
> > > +++ b/libavformat/webpenc.c
> > > @@ -24,10 +24,20 @@
> > >  #include "avformat.h"
> > >  #include "internal.h"
> > >
> > > +#include <webp/encode.h>
> > > +#if (WEBP_ENCODER_ABI_VERSION >= 0x0206)
> > > +#include <webp/mux.h>
> > > +#if (WEBP_MUX_ABI_VERSION >= 0x0104)
> > > +#define USE_WEBP_ANIMENCODER
> > > +#endif
> > > +#endif
> >
> > While checking for CONFIG_LIBWEBP would be enough to know if
> webp/encode.h
> > is available
> > here as i mentioned in a previous email, it doesn't guarantee that
> > webp/mux.h is available
> > as well, so you will need to add webp_mux_h to HEADERS_LIST in configure
> > (the check for
> > libwebpmux you already added will enable it if available) and use the
> > resulting define.
> >
>
> thanks for the suggestions, i'll rework and post an updated patch soon.
>
> skal/
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg_animated_webp.git.diff
Type: text/x-patch
Size: 22718 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150416/51880535/attachment.bin>