diff mbox series

[FFmpeg-devel,13/19] avutil/samplefmt: Don't include attributes.h, avutil.h

Message ID AM7PR03MB666086679F313F512FEB09378F349@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit 8e7d2e46211f932b3511807bdd3c951eb95672aa
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:43 p.m. UTC
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavutil/samplefmt.c | 6 ++++--
 libavutil/samplefmt.h | 3 ---
 2 files changed, 4 insertions(+), 5 deletions(-)

Comments

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

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

LGTM

// Martin
diff mbox series

Patch

diff --git a/libavutil/samplefmt.c b/libavutil/samplefmt.c
index c7428940e1..6d3ec34dab 100644
--- a/libavutil/samplefmt.c
+++ b/libavutil/samplefmt.c
@@ -16,11 +16,13 @@ 
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#include "common.h"
+#include "error.h"
+#include "macros.h"
+#include "mem.h"
 #include "samplefmt.h"
 
+#include <limits.h>
 #include <stdio.h>
-#include <stdlib.h>
 #include <string.h>
 
 typedef struct SampleFmtInfo {
diff --git a/libavutil/samplefmt.h b/libavutil/samplefmt.h
index f270199195..1999c9bca6 100644
--- a/libavutil/samplefmt.h
+++ b/libavutil/samplefmt.h
@@ -21,9 +21,6 @@ 
 
 #include <stdint.h>
 
-#include "avutil.h"
-#include "attributes.h"
-
 /**
  * @addtogroup lavu_audio
  * @{