[FFmpeg-devel] [PATCH] avutil/attributes: add av_warn_unused_result

Ganesh Ajjanagadde gajjanagadde at gmail.com
Mon Oct 5 07:45:12 CEST 2015

GCC 3.4 introduced an attribute warn_unused_result to warn when a programmer
discards the return value. Applying this judiciously across the codebase can help
in fixing a lot of problems. At a high level, functions which return error codes
should always be checked. More concretely, consider the functions ff_add_format
and the like in avfilter/formats.h. A quick examination shows that a large portion
of libavfilter fails to handle the associated errors, usually AVERROR(ENOMEM).
The above example was where I observed the utility of this, but it should be
useful in many places across the code base.

Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
 libavutil/attributes.h | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libavutil/attributes.h b/libavutil/attributes.h
index b4b5f13..3b94466 100644
--- a/libavutil/attributes.h
+++ b/libavutil/attributes.h
@@ -52,6 +52,12 @@
+#    define av_warn_unused_result __attribute__((warn_unused_result))
+#    define av_warn_unused_result
 #    define av_noinline __attribute__((noinline))
 #elif defined(_MSC_VER)

More information about the ffmpeg-devel mailing list