New API in kvazaar 0.7 (doesn't build with --enable-libkvazaar)

[marillat]

In kvazaar 0.7 the API has been changed in encoder_encode() and thus ffmpeg doesn't build with --enable-libkvazaar with the following error :

CC	libavcodec/libkvazaar.o
libavcodec/libkvazaar.c: In function ‘libkvazaar_encode’:
libavcodec/libkvazaar.c:176:10: error: too few arguments to function ‘ctx->api->encoder_encode’
     if (!ctx->api->encoder_encode(ctx->encoder, img_in, &data_out, &len_out, NULL)) {
          ^
libavcodec/libkvazaar.c:186:9: warning: ‘ff_alloc_packet’ is deprecated [-Wdeprecated-declarations]
         retval = ff_alloc_packet(avpkt, len_out);
         ^
In file included from libavcodec/libkvazaar.c:30:0:
libavcodec/internal.h:227:33: note: declared here
 attribute_deprecated int ff_alloc_packet(AVPacket *avpkt, int size);
                                 ^
common.mak:57: recipe for target 'libavcodec/libkvazaar.o' failed

[Carl Eugen Hoyos]

What FFmpeg version are you testing?

[marillat]

Replying to cehoyos:

What FFmpeg version are you testing?

2.8 and 2.8.1

[Carl Eugen Hoyos]

Untested patch sent.

[Carl Eugen Hoyos]

Hopefully fixed in c2db8ebc083e5fa35def204321a225960beede84

[marillat]

Replying to cehoyos:

Untested patch sent.

The patch is the one in c2db8ebc083e5fa35def204321a225960beede84 ?

[Carl Eugen Hoyos]

Yes.

[marillat]

Maybe not the best way to fix a bug report by disabling feature, especially when git master is already ported to kvazaar >= 0.7

[Carl Eugen Hoyos]

I suspect this is how a release series is defined: Changing the version of an external dependency can't be ok...