aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2014-09-08 13:49:36 +0200
committerFlorian Weimer <fweimer@redhat.com>2014-09-10 08:13:06 +0200
commite6fb95871cd3557e5882a6f969c11bc6a3cbe781 (patch)
treee5aed138cf0ff10eeea2feb824f9fc3af7e2d5b2
parent9d3a350b4951351b3d821e8138293470de12abb2 (diff)
downloadglibc-e6fb95871cd3557e5882a6f969c11bc6a3cbe781.zip
glibc-e6fb95871cd3557e5882a6f969c11bc6a3cbe781.tar.gz
glibc-e6fb95871cd3557e5882a6f969c11bc6a3cbe781.tar.bz2
Turn on -Werror=implicit-function-declaration
GCC 4.4, the minimum compiler version, supports this option. Unlike other warnings, -Wimplicit-function-declaration warnings should be independent of compiler versions, so this change should not cause compiler-specific build failures.
-rw-r--r--ChangeLog8
-rw-r--r--Makeconfig2
-rw-r--r--stdlib/tst-tls-atexit-lib.c2
3 files changed, 11 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 63efb00..cb8aeb7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2014-09-10 Florian Weimer <fweimer@redhat.com>
+
+ * Makeconfig (gccwarn-c): Turn implicit-function-declaration
+ warnings into errors.
+
+ * stdlib/tst-tls-atexit-lib.c: Include <stdlib.h>, for the
+ __cxa_thread_atexit_impl prototype.
+
2014-09-09 Steve Ellcey <sellcey@mips.com>
* sysdeps/mips/preconfigure: Put code inside mips* case statement.
diff --git a/Makeconfig b/Makeconfig
index cef0f06..0f8069b 100644
--- a/Makeconfig
+++ b/Makeconfig
@@ -717,7 +717,7 @@ else
+gccwarn := -Wall -Wwrite-strings -Winline
endif
+gccwarn += -Wundef
-+gccwarn-c = -Wstrict-prototypes
++gccwarn-c = -Wstrict-prototypes -Werror=implicit-function-declaration
# We do not depend on the address of constants in different files to be
# actually different, so allow the compiler to merge them all.
diff --git a/stdlib/tst-tls-atexit-lib.c b/stdlib/tst-tls-atexit-lib.c
index 964f94b..b67d62f 100644
--- a/stdlib/tst-tls-atexit-lib.c
+++ b/stdlib/tst-tls-atexit-lib.c
@@ -16,6 +16,8 @@
License along with the GNU C Library; if not, see
<http://www.gnu.org/licenses/>. */
+#include <stdlib.h>
+
extern void *__dso_handle;
typedef struct