On Mon, Sep 25, 2017 at 11:13 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 25/09/2017 07:40, Rohit Bansal wrote: > > The patch looks nice and thank you for the minimal sample file. > > If nobody is against this I'll edit the commit message to fit the > commit guidelines https://wiki.libav.org/GitCommit and push it. >
The check seems a bit odd. Why include "id" or "crc_abs" in it? Both can be 0/1 in valid files. Whats with the bit-shfiting shenanigans? If you just want to check if layer is 0, just check if layer is 0? - Hendrik _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel