Hi, 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. >
I agree, though for this if you wanted compatibility, since the values are defined by the bitstream, using 0/1 would be enough; the enum is merely cosmetic. >> >> 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