From: Simon Marchi Date: Fri, 27 Mar 2020 01:11:35 +0000 (-0400) Subject: Fix: python: suppress -Wmissing-prototypes warning with SWIG 3.0.10 X-Git-Tag: v2.13.0-rc1~680 X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=862e39082c67c68876493ae73845db5be521fb75 Fix: python: suppress -Wmissing-prototypes warning with SWIG 3.0.10 SWIG 3.0.10 is used on SLES 12. It produces this warning: CC lttng_wrap.lo lttng_wrap.c:3411:1: error: no previous prototype for ‘SWIG_strnlen’ [-Werror=missing-prototypes] 3411 | SWIG_strnlen(const char* s, size_t maxlen) | ^~~~~~ SWIG_strnlen is defined like this: size_t SWIG_strnlen(const char* s, size_t maxlen) { ... } Since the function is not static and has no previous declaration, the diagnostic is emitted. We can see that they have fixed it in SWIG 3.0.12, where the same function is defined as: SWIGINTERN size_t SWIG_strnlen(const char* s, size_t maxlen) { ... } SWIGINTERN is defined as static. We can workaround the warning by adding our own declaration for SWIG_strnlen in lttng.i(.in). Tested by build with SWIG 3.0.10, 3.0.11 and 3.0.12. Change-Id: I7d9d93cf5fc04f2044a47ad384e16d726e27f72f Signed-off-by: Simon Marchi Signed-off-by: Jérémie Galarneau --- diff --git a/extras/bindings/swig/python/lttng.i.in b/extras/bindings/swig/python/lttng.i.in index b6b6f960e..684482e1f 100644 --- a/extras/bindings/swig/python/lttng.i.in +++ b/extras/bindings/swig/python/lttng.i.in @@ -32,6 +32,11 @@ multiple concurrent processes and threads. Tracing across multiple systems is al // Avoid -Wmissing-declarations warning. PyObject *SWIG_init(void); + +// Avoid -Wmissing-prototypes warning with SWIG 3.0.10. +#if SWIG_VERSION < 0x30012 +static size_t SWIG_strnlen(const char* s, size_t maxlen); +#endif %} typedef unsigned int uint32_t;