>> --- 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