[libav-devel] [PATCH 7/8] checkasm: add HEVC MC tests

Henrik Gramner henrik at gramner.com
Thu Aug 20 22:37:02 CEST 2015


On Wed, Aug 19, 2015 at 9:43 PM, Anton Khirnov <anton at khirnov.net> wrote:
> +    const int srcstride = FFALIGN(width, 16) * sizeof(*src0);
> +    const int dststride = FFALIGN(width, 16) * PIXEL_SIZE(bit_depth);

 Strides, and any other pointer-sized value, should be ptrdiff_t - or
more preferable, review/push my checkasm patch and rebase this one on
top of that to get rid of the issue. :)

> +        report("%s", "qpel");
> +        report("%s", "epel");
> +        report("%s", "unweighted_pred");
> +        report("%s", "weighted_pred");

The "%s" is redundant with string literals.


More information about the libav-devel mailing list