[FFmpeg-devel] [PATCH] libvpx: Support setting color range for vp9.

[Sasi Inguva]

Sorry for the delay on this. Was waiting for
https://chromium-review.googlesource.com/#/c/320116/ to be submitted. Sent
the new patch with decoder changes.

On Mon, Jan 4, 2016 at 12:26 PM, James Almer <jamrial at gmail.com> wrote:

> On 1/4/2016 5:01 PM, Sasi Inguva wrote:
> > Would it be a lot easier and correct if I just update the
> IMAGE_ABI_VERSION to 4 in libvpx HEAD and check here in the decoder
> IMAGE_ABI_VERSION > 3  ?
>
> Yes, bumping any of the defines would probably be best. It will not apply
> to libvpxdec 1.5.0, though, but i guess it's better than bloating configure
> with checks for specific struct members.
>
> >
> > On Mon, Jan 4, 2016 at 11:03 AM, Ronald S. Bultje <rsbultje at gmail.com
> <mailto:rsbultje at gmail.com>> wrote:
> >
> >     Hi,
> >
> >     On Mon, Jan 4, 2016 at 1:48 PM, Sasi Inguva <isasi at google.com
> <mailto:isasi at google.com>> wrote:
> >
> >     > Ping! I could not find any other fields for version other than the
> >     > IMAGE_ABI_VERSION , CODEC_ABI_VERSION, DECODER_ABI_VERSION which
> remain
> >     > unchanged .
> >
> >
> >     So, add a version check? Or check for the existence of the
> appropriate
> >     color range field in this struct in our configure script.
> >
> >     Ronald
> >     _______________________________________________
> >     ffmpeg-devel mailing list
> >     ffmpeg-devel at ffmpeg.org <mailto:ffmpeg-devel at ffmpeg.org>
> >     http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> >
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>