diff mbox series

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

Message ID AM7PR03MB6660671D80CBF58A7F5C62798F349@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit e82d94ee469944344ad63dc4ee8cdc6c88613e57
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 
It is a remnant of an FF_API_* inclusion (back from when they were in
avutil.h and not in version.h).

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavutil/imgutils.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

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

> It is a remnant of an FF_API_* inclusion (back from when they were in
> avutil.h and not in version.h).
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> libavutil/imgutils.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)

LGTM

// Martin
diff mbox series

Patch

diff --git a/libavutil/imgutils.h b/libavutil/imgutils.h
index cb2d74728e..be53335568 100644
--- a/libavutil/imgutils.h
+++ b/libavutil/imgutils.h
@@ -27,8 +27,10 @@ 
  * @{
  */
 
-#include "avutil.h"
+#include <stddef.h>
+#include <stdint.h>
 #include "pixdesc.h"
+#include "pixfmt.h"
 #include "rational.h"
 
 /**