diff mbox series

[FFmpeg-devel,16/19] avutil/display: Don't include avutil.h

Message ID AM7PR03MB6660D9FAA422C5FFECA979498F349@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit b389ff9a8588d603215ce5714e6c17b4882c22d6
Headers show
Series [FFmpeg-devel] avutil/x86/emms: Don't unnecessarily include lavu/cpu.h | expand

Checks

Context Check Description
yinshiyou/make_fate_loongarch64 success Make fate finished
yinshiyou/make_loongarch64 warning New warnings during build
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Andreas Rheinhardt Feb. 15, 2022, 5:43 p.m. UTC
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavutil/display.c       | 1 +
 libavutil/display.h       | 1 -
 libavutil/tests/display.c | 1 +
 3 files changed, 2 insertions(+), 1 deletion(-)

Comments

Martin Storsjö Feb. 23, 2022, 2:52 p.m. UTC | #1
On Tue, 15 Feb 2022, Andreas Rheinhardt wrote:

> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> libavutil/display.c       | 1 +
> libavutil/display.h       | 1 -
> libavutil/tests/display.c | 1 +
> 3 files changed, 2 insertions(+), 1 deletion(-)

LGTM

// Martin
diff mbox series

Patch

diff --git a/libavutil/display.c b/libavutil/display.c
index a0076e067b..d31061283c 100644
--- a/libavutil/display.c
+++ b/libavutil/display.c
@@ -23,6 +23,7 @@ 
 #include <math.h>
 
 #include "display.h"
+#include "libm.h"
 #include "mathematics.h"
 
 // fixed point to double
diff --git a/libavutil/display.h b/libavutil/display.h
index d87bf68425..31d8bef361 100644
--- a/libavutil/display.h
+++ b/libavutil/display.h
@@ -27,7 +27,6 @@ 
 #define AVUTIL_DISPLAY_H
 
 #include <stdint.h>
-#include "common.h"
 
 /**
  * @addtogroup lavu_video
diff --git a/libavutil/tests/display.c b/libavutil/tests/display.c
index 893ebb5543..19b07fc15f 100644
--- a/libavutil/tests/display.c
+++ b/libavutil/tests/display.c
@@ -18,6 +18,7 @@ 
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
+#include <stdio.h>
 #include "libavutil/display.c"
 
 static void print_matrix(int32_t matrix[9])