Hi, On Mon, Jan 4, 2016 at 12:26 PM, James Almer <[email protected]> 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 <[email protected] >> <mailto:[email protected]>> wrote: >> >> Hi, >> >> On Mon, Jan 4, 2016 at 1:48 PM, Sasi Inguva <[email protected] >> <mailto:[email protected]>> 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 >> [email protected] <mailto:[email protected]> >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> >> > > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
