diff mbox series

[FFmpeg-devel,08/19] avutil/audio_fifo: Avoid avutil.h inclusion

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

Checks

Context Check Description
yinshiyou/configure_loongarch64 warning Failed to run configure
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:42 p.m. UTC
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavutil/audio_fifo.c | 7 +++++--
 libavutil/audio_fifo.h | 2 +-
 2 files changed, 6 insertions(+), 3 deletions(-)

Comments

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

> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> libavutil/audio_fifo.c | 7 +++++--
> libavutil/audio_fifo.h | 2 +-
> 2 files changed, 6 insertions(+), 3 deletions(-)

LGTM

// Martin
diff mbox series

Patch

diff --git a/libavutil/audio_fifo.c b/libavutil/audio_fifo.c
index b1355e55a0..f4103178ba 100644
--- a/libavutil/audio_fifo.c
+++ b/libavutil/audio_fifo.c
@@ -24,10 +24,13 @@ 
  * Audio FIFO
  */
 
-#include "avutil.h"
+#include <limits.h>
+#include <stddef.h>
+
 #include "audio_fifo.h"
-#include "common.h"
+#include "error.h"
 #include "fifo.h"
+#include "macros.h"
 #include "mem.h"
 #include "samplefmt.h"
 
diff --git a/libavutil/audio_fifo.h b/libavutil/audio_fifo.h
index 9d570b04c0..d1e4c856dc 100644
--- a/libavutil/audio_fifo.h
+++ b/libavutil/audio_fifo.h
@@ -27,7 +27,7 @@ 
 #ifndef AVUTIL_AUDIO_FIFO_H
 #define AVUTIL_AUDIO_FIFO_H
 
-#include "avutil.h"
+#include "attributes.h"
 #include "samplefmt.h"
 
 /**