diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2015-10-21 15:14:13 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2015-10-21 15:14:13 +0000 |
| commit | 8e2f7c68111ba8fae795ee08b6822e21ab8cfa38 (patch) | |
| tree | b05f1b50eba10b59ad91da500dcfecc1d9d88334 | |
| parent | 85231522bb178ebe0957529de20b7f3dda81168c (diff) | |
| download | glibc-8e2f7c68111ba8fae795ee08b6822e21ab8cfa38.tar glibc-8e2f7c68111ba8fae795ee08b6822e21ab8cfa38.tar.gz glibc-8e2f7c68111ba8fae795ee08b6822e21ab8cfa38.tar.bz2 glibc-8e2f7c68111ba8fae795ee08b6822e21ab8cfa38.zip | |
Use -Wold-style-definition.
This patch makes glibc build with -Wold-style-definition to avoid
old-style function definitions creeping back in by accident.
Tested for x86_64 and x86 (testsuite, and that installed shared
libraries are unchanged by the patch). Also tested build for arm,
mips64 and powerpc32. Hopefully there aren't any remaining
system-specific files with old-style definitions whose formatting
evaded my searches, but if there are, they will be easy to fix.
* Makeconfig (+gccwarn-c): Add -Wold-style-definition.
* Makefile ($(objpfx)c++-types-check.out): Filter out
$(+gccwarn-c) instead of -Wstrict-prototypes.
| -rw-r--r-- | ChangeLog | 4 | ||||
| -rw-r--r-- | Makeconfig | 2 | ||||
| -rw-r--r-- | Makefile | 2 |
3 files changed, 6 insertions, 2 deletions
@@ -1,5 +1,9 @@ 2015-10-21 Joseph Myers <joseph@codesourcery.com> + * Makeconfig (+gccwarn-c): Add -Wold-style-definition. + * Makefile ($(objpfx)c++-types-check.out): Filter out + $(+gccwarn-c) instead of -Wstrict-prototypes. + * io/fts.c (fts_open): Convert to prototype-style function definition. * malloc/mcheck.c (mcheck): Likewise. diff --git a/Makeconfig b/Makeconfig index 1dc522c7ed..f3668c616a 100644 --- a/Makeconfig +++ b/Makeconfig @@ -752,7 +752,7 @@ endif ifeq ($(enable-werror),yes) +gccwarn += -Werror endif -+gccwarn-c = -Wstrict-prototypes ++gccwarn-c = -Wstrict-prototypes -Wold-style-definition # We do not depend on the address of constants in different files to be # actually different, so allow the compiler to merge them all. @@ -256,7 +256,7 @@ ifneq ($(CXX),no) vpath c++-types.data $(+sysdep_dirs) $(objpfx)c++-types-check.out: c++-types.data scripts/check-c++-types.sh - scripts/check-c++-types.sh $< $(CXX) $(filter-out -std=gnu99 -Wstrict-prototypes,$(CFLAGS)) $(CPPFLAGS) > $@; \ + scripts/check-c++-types.sh $< $(CXX) $(filter-out -std=gnu99 $(+gccwarn-c),$(CFLAGS)) $(CPPFLAGS) > $@; \ $(evaluate-test) endif |