[FFmpeg-devel] [PATCH 1/5] avcodec/tableprint: Don't include mem_internal.h
Andreas Rheinhardt
andreas.rheinhardt at gmail.com
Tue Jan 12 02:14:35 EET 2021
Andreas Rheinhardt:
> tableprint.h does not declare anything as aligned; it just prints
> DECLARE_ALIGNED. So it can be removed; in fact, it needs to be removed,
> because mem_internal.h includes config.h which leads to warnings when
> building with hardcoded tables enabled because of redefinitions of
> CONFIG_HARDCODED_TABLES.
>
> (Furthermore, config.h is only valid for the target, not the host,
> so HAVE_LOCAL_ALIGNED might even be wrong here.)
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> libavcodec/tableprint.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/libavcodec/tableprint.h b/libavcodec/tableprint.h
> index e57eeb6ca6..6f61c7124b 100644
> --- a/libavcodec/tableprint.h
> +++ b/libavcodec/tableprint.h
> @@ -27,7 +27,6 @@
> #include <stdio.h>
>
> #include "libavutil/common.h"
> -#include "libavutil/mem_internal.h"
>
> #define WRITE_1D_FUNC_ARGV(type, linebrk, fmtstr, ...)\
> void write_##type##_array(const type *data, int len)\
>
Will apply this patch tomorrow unless there are objections.
- Andreas
More information about the ffmpeg-devel
mailing list