configure: enable -Wstrict-prototypes unconditionally
authorSimon Marchi <simon.marchi@efficios.com>
Wed, 15 Jul 2020 16:17:24 +0000 (12:17 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Thu, 16 Jul 2020 22:39:41 +0000 (18:39 -0400)
I noticed that -Wstrict-prototypes would not get enabled, because the
autoconf test program generates a warning of this type.  Enable it
unconditionally, like we already do for -Wold-style-definition.

Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I145ceedb977d1d60a094f598ecae56930c742c4c

configure.ac

index 4e3a07a26ed4c17ea223af67762214c81bd063a3..63eef1bcf611f16ff7103c8f0cb891f30882fdfc 100644 (file)
@@ -666,7 +666,6 @@ CFLAGS=${save_CFLAGS}
 AX_APPEND_COMPILE_FLAGS([ dnl
                -Wall dnl
                -Wextra dnl
-               -Wstrict-prototypes dnl
                -Wmissing-prototypes dnl
                -Wmissing-declarations dnl
                -Wnull-dereference dnl
@@ -714,15 +713,13 @@ AS_IF([test "x$enable_Werror" = "xyes"],
 )
 
 # The test used in AX_APPEND_COMPILE_FLAGS, generated using AC_LANG_PROGRAM, is
-# written in such a way that it triggers a -Wold-style-definition warning.  So
-# this warning always ends up disabled if we put it there, because the test
-# program does not build.
+# written in such a way that it triggers warnings with the following warning
+# flags.  So they would always end up disabled if we put them there, because
+# the test program would not build.
 #
-# Enable it here unconditionally.  It is supported by GCC >= 4.8 and by Clang
-# (it is accepted for compatibility although it has no effect), and there is
-# not reason to not want it.
-
-WARN_CFLAGS="${WARN_CFLAGS} -Wold-style-definition"
+# Enable them here unconditionally.  They are supported by GCC >= 4.8 and by
+# Clang >= 4.0.
+WARN_CFLAGS="${WARN_CFLAGS} -Wold-style-definition -Wstrict-prototypes"
 
 # CFLAGS from AX_APPEND_COMPILE_FLAGS.
 AM_CFLAGS="${AM_CFLAGS} ${WARN_CFLAGS}"
This page took 0.025419 seconds and 4 git commands to generate.