From: Simon Marchi Date: Wed, 25 Mar 2020 22:40:02 +0000 (-0400) Subject: configure: add -Wmissing-declarations, -Wmissing-prototypes, and more X-Git-Tag: v2.13.0-rc1~704 X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=33cca43b7056157c19d720a90e0d9e11bef2b83e;ds=inline configure: add -Wmissing-declarations, -Wmissing-prototypes, and more Here's the rationale for each: - -Wmissing-declarations: Make sure the definition of a function can "see" a corresponding (usually in a header file), if it isn't static. This makes sure that the declaration and definition don't go out of sync, which can lead to hard to debug problems (because it still builds, but the function doesn't receives what it thinks it receives). On top of pointing out out-of-sync declarations, it can help point out that a foo.c file misses including its header foo.c, or that a function should actually be made static. - -Wmissing-prototypes: makes sure that functions without parameters are declared as `foo(void)` instead of `foo()`. In C, the former declares a function that takes no parameters, whereas the latter declares a function without specifying its parameters. The latter could be called with any number of parameters, which is a recipe for confusion. - -Wmissing-parameter-type, -Wold-style-definition, -Wold-style-declarations, -Wstrict-prototypes: makes sure there's no function declared with parameters without types specified, or using the old style: int foo(bar) int bar; { ... } Unlikely, but there's no harm in enabling them. Change-Id: I7ddf5ff61b4466c0bd7b03485ef29156c399e2a8 Signed-off-by: Simon Marchi Signed-off-by: Jérémie Galarneau --- diff --git a/configure.ac b/configure.ac index a997ba3b9..4e186d471 100644 --- a/configure.ac +++ b/configure.ac @@ -39,6 +39,11 @@ AX_APPEND_COMPILE_FLAGS([ dnl dnl ^ -Wno-incomplete-setjmp-declaration dnl -Wdiscarded-qualifiers dnl + -Wmissing-declarations dnl + -Wmissing-prototypes dnl + -Wmissing-parameter-type dnl + -Wold-style-definition dnl + -Wstrict-prototypes dnl ], [WARN_CFLAGS], [-Werror])