diff mbox series

[FFmpeg-devel,2/2] lavu/avassert: include config.h

Message ID 20230426123138.135521-2-george@nsup.org
State New
Headers show
Series [FFmpeg-devel,1/2] lavc/intrax8: fix an assert | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Nicolas George April 26, 2023, 12:31 p.m. UTC 
Fix setting the assert level.

Signed-off-by: Nicolas George <george@nsup.org>
---
 libavutil/avassert.h | 1 +
 1 file changed, 1 insertion(+)

Comments

Hendrik Leppkes April 26, 2023, 1:05 p.m. UTC | #1 
On Wed, Apr 26, 2023 at 2:32 PM Nicolas George <george@nsup.org> wrote:
>
> Fix setting the assert level.
>
> Signed-off-by: Nicolas George <george@nsup.org>
> ---
>  libavutil/avassert.h | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/libavutil/avassert.h b/libavutil/avassert.h
> index 51e462bbae..8f3f72c80c 100644
> --- a/libavutil/avassert.h
> +++ b/libavutil/avassert.h
> @@ -28,6 +28,7 @@
>  #define AVUTIL_AVASSERT_H
>
>  #include <stdlib.h>
> +#include "config.h"
>  #include "log.h"
>  #include "macros.h"
>

This is an installed header, it cannot depend on config.h

- Hendrik
Nicolas George April 26, 2023, 1:17 p.m. UTC | #2 
Hendrik Leppkes (12023-04-26):
> This is an installed header, it cannot depend on config.h

Thanks for spotting it, that was counter-intuitive since config.h is
almost indispensable to its workings.

Updated version attached, similar to what exists in common.h.

Regards,
Nicolas George July 12, 2023, 2:05 p.m. UTC | #3 
Nicolas George (12023-04-26):
> Updated version attached, similar to what exists in common.h.

Pushed.

Regards,
diff mbox series

Patch

diff --git a/libavutil/avassert.h b/libavutil/avassert.h
index 51e462bbae..8f3f72c80c 100644
--- a/libavutil/avassert.h
+++ b/libavutil/avassert.h
@@ -28,6 +28,7 @@ 
 #define AVUTIL_AVASSERT_H
 
 #include <stdlib.h>
+#include "config.h"
 #include "log.h"
 #include "macros.h"