[FFmpeg-devel] [PATCH 1/2] checkasm: Fix compilation with --disable-avcodec

[Ronald S. Bultje]

Hi,

On Thu, Oct 1, 2015 at 1:50 PM, Henrik Gramner <henrik at gramner.com> wrote:

> ---
>  tests/checkasm/checkasm.c | 42 ++++++++++++++++++++++--------------------
>  1 file changed, 22 insertions(+), 20 deletions(-)
>
> diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c
> index d7c2e8a..264473a 100644
> --- a/tests/checkasm/checkasm.c
> +++ b/tests/checkasm/checkasm.c
> @@ -57,26 +57,28 @@ static const struct {
>      const char *name;
>      void (*func)(void);
>  } tests[] = {
> -#if CONFIG_BSWAPDSP
> -    { "bswapdsp", checkasm_check_bswapdsp },
> -#endif
> -#if CONFIG_FLACDSP
> -    { "flacdsp", checkasm_check_flacdsp },
> -#endif
> -#if CONFIG_H264PRED
> -    { "h264pred", checkasm_check_h264pred },
> -#endif
> -#if CONFIG_H264QPEL
> -    { "h264qpel", checkasm_check_h264qpel },
> -#endif
> -#if CONFIG_JPEG2000_DECODER
> -    { "jpeg2000dsp", checkasm_check_jpeg2000dsp },
> -#endif
> -#if CONFIG_V210_ENCODER
> -    { "v210enc", checkasm_check_v210enc },
> -#endif
> -#if CONFIG_VP9_DECODER
> -    { "vp9dsp", checkasm_check_vp9dsp },
> +#if CONFIG_AVCODEC
> +    #if CONFIG_BSWAPDSP
> +        { "bswapdsp", checkasm_check_bswapdsp },
> +    #endif
> +    #if CONFIG_FLACDSP
> +        { "flacdsp", checkasm_check_flacdsp },
> +    #endif
> +    #if CONFIG_H264PRED
> +        { "h264pred", checkasm_check_h264pred },
> +    #endif
> +    #if CONFIG_H264QPEL
> +        { "h264qpel", checkasm_check_h264qpel },
> +    #endif
> +    #if CONFIG_JPEG2000_DECODER
> +        { "jpeg2000dsp", checkasm_check_jpeg2000dsp },
> +    #endif
> +    #if CONFIG_V210_ENCODER
> +        { "v210enc", checkasm_check_v210enc },
> +    #endif
> +    #if CONFIG_VP9_DECODER
> +        { "vp9dsp", checkasm_check_vp9dsp },
> +    #endif
>  #endif
>      { NULL }
>  };
> --
> 1.9.1


Does this mean these macros are defined and 1 when AVCODEC=0?

That seems like a bug?

Ronald