diff mbox series

[FFmpeg-devel,3/3] libavutil/log.c: only include valgrind header when used.

Message ID 20231029175657.46386-3-Reimar.Doeffinger@gmx.de
State Accepted
Commit 9dd49c8b52c3f6f5a486764fce42ec5d5fac8d16
Headers show
Series [FFmpeg-devel,1/3] configure: fix _Pragma check. | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Reimar Döffinger Oct. 29, 2023, 5:56 p.m. UTC
From: Reimar Döffinger <Reimar.Doeffinger@gmx.de>

This is cleaner, but it is also a workaround for when
the header exists, but cannot be compiled.
This will happen when the compiler has no inline asm
support.
Possibly the configure check should be improved as well.
---
 libavutil/log.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/libavutil/log.c b/libavutil/log.c
index 5948e50467..2d358b7ab9 100644
--- a/libavutil/log.c
+++ b/libavutil/log.c
@@ -47,7 +47,7 @@  static AVMutex mutex = AV_MUTEX_INITIALIZER;
 
 #define LINE_SZ 1024
 
-#if HAVE_VALGRIND_VALGRIND_H
+#if HAVE_VALGRIND_VALGRIND_H && CONFIG_VALGRIND_BACKTRACE
 #include <valgrind/valgrind.h>
 /* this is the log level at which valgrind will output a full backtrace */
 #define BACKTRACE_LOGLEVEL AV_LOG_ERROR