>> --- a/tests/checkasm/checkasm.h
>> +++ b/tests/checkasm/checkasm.h
>> @@ -38,6 +38,7 @@ void checkasm_check_h264dsp(void);
>>  void checkasm_check_h264qpel(void);
>>  void checkasm_check_hevc_mc(void);
>> +void checkasm_check_hevc_idct(void);
>>  void checkasm_check_synth_filter(void);
>>  void checkasm_check_v210enc(void);
>
> Order all of these.

I'll place hevc_idct before hevc_mc
>
>> --- /dev/null
>> +++ b/tests/checkasm/hevc_idct.c
>> @@ -0,0 +1,75 @@
>> +
>> +#include <string.h>
>> +
>> +#include "checkasm.h"
>> +
>> +#include "libavcodec/avcodec.h"
>> +#include "libavcodec/hevcdsp.h"
>> +
>> +#include "libavutil/common.h"
>> +#include "libavutil/intreadwrite.h"
>
> The order of the include blocks is reserved.
>
> I only see hevcdsp.h being required.

intreadwrite.h is needed for AV_WN, but common.h and avcodec.h should
be skipped
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to