On Mon, Sep 05, 2016 at 01:02:40PM +0200, Anton Khirnov wrote:
> --- /dev/null
> +++ b/tests/checkasm/audiodsp.c
> @@ -0,0 +1,139 @@
> +
> +#include <math.h>
> +#include <string.h>
> +#include <stdio.h>
> +#include <stdlib.h>

nit: order

> +#include "libavcodec/audiodsp.h"
> +
> +#include "libavutil/common.h"
> +#include "libavutil/intreadwrite.h"
> +
> +#include "checkasm.h"

libavutil headers before libavcodec headers

> +        val1 = ((int32_t)rnd());
> +        val2 = ((int32_t)rnd());

unnecessary ()

probably OK

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to