[FFmpeg-devel] [PATCH 1/5] avcodec/tableprint: Don't include mem_internal.h

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Thu Jan 7 01:13:04 EET 2021


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)\
-- 
2.25.1



More information about the ffmpeg-devel mailing list