On Sat, Sep 02, 2017 at 10:24:45PM +0300, Martin Storsjö wrote:
> ---
> tests/checkasm/h264dsp.c | 90
> +++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 89 insertions(+), 1 deletion(-)
> --- a/tests/checkasm/h264dsp.c
> +++ b/tests/checkasm/h264dsp.c
> @@ -223,10 +224,97 @@ static void check_idct(void)
> +
> +static void check_idct_multiple(void)
> +{
> + LOCAL_ALIGNED_16(uint8_t, dst_full, [16 * 16 * 2]);
> + LOCAL_ALIGNED_16(int16_t, coef_full, [16 * 16 * 2]);
nit: slightly odd spacing
> + LOCAL_ALIGNED_16(uint8_t, dst0, [16 * 16 * 2]);
> + LOCAL_ALIGNED_16(uint8_t, dst1, [16 * 16 * 2]);
> + LOCAL_ALIGNED_16(int16_t, coef0, [16 * 16 * 2]);
> + LOCAL_ALIGNED_16(int16_t, coef1, [16 * 16 * 2]);
> + LOCAL_ALIGNED_16(uint8_t, nnzc, [15*8]);
nit: spaces around *
> +
> + if (nnz > 1)
> + nnz = sz*sz;
> + memcpy(&coef_full[i*SIZEOF_COEF/sizeof(coef[0])],
nit: spaces around *
Patch should be OK.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
