On Wed, Sep 21, 2011 at 02:31:00PM +0100, Mans Rullgard wrote:
> This fixes some nonsensical code by moving some should-be-shared
> code to separate functions and using these.
> 
> Signed-off-by: Mans Rullgard <[email protected]>
> ---
>  libavcodec/dct-test.c |  130 
> +++++++++++++++++++++----------------------------
>  1 files changed, 55 insertions(+), 75 deletions(-)
> 
> diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c
> index 962b370..3d2dab4 100644
> --- a/libavcodec/dct-test.c
> +++ b/libavcodec/dct-test.c
> @@ -199,6 +199,56 @@ static inline void mmx_emms(void)
>  #endif
>  }
>  
> +static void init_block(DCTELEM block[64], int test, int is_idct, AVLFG *prng)
> +{
> +    int i, j;
> +
> +    memset(block, 0, 64 * sizeof(block));

ahem, that's too much

Otherwise looks OK
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to