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 <jamr...@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 <rsbul...@gmail.com > <mailto:rsbul...@gmail.com>> wrote: > > > > Hi, > > > > On Mon, Jan 4, 2016 at 1:48 PM, Sasi Inguva <is...@google.com > <mailto:is...@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@ffmpeg.org <mailto:ffmpeg-devel@ffmpeg.org> > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
