Mercurial > hg > xemacs-beta
diff configure.in @ 2315:e219f63b9c24
[xemacs-hg @ 2004-09-29 15:38:30 by james]
Turn of -Wunused-parameter for g++.
author | james |
---|---|
date | Wed, 29 Sep 2004 15:38:32 +0000 |
parents | e2fdc9035c5b |
children | ecf1ebac70d8 |
line wrap: on
line diff
--- a/configure.in Tue Sep 28 21:51:51 2004 +0000 +++ b/configure.in Wed Sep 29 15:38:32 2004 +0000 @@ -2059,9 +2059,7 @@ dnl Yuck, bad compares have been worth at least 3 crashes! cflags_warning="$cflags_warning -Wsign-compare" cflags_warning="$cflags_warning -Wundef -Wstrict-prototypes" - if test "$__GCC3" = "yes"; then - cflags_warning="$cflags_warning -Wpacked -Wunused-parameter -Wunused-variable" - fi + test "$__GCC3" = "yes" && cflags_warning="$cflags_warning -Wpacked" dnl glibc is intentionally not `-Wpointer-arith'-clean. dnl Ulrich Drepper has rejected patches to fix the glibc header files. test "$have_glibc" != "yes" && \ @@ -2073,8 +2071,13 @@ dnl complaints about parameters with the same names as global functions. dnl There is no -Wmissing-declarations under g++. dnl But gcc warns about -Weffc++ in C compiles. - test "$xemacs_compiler" = "g++" && \ + dnl With g++, there is no effective way to use -Wunused-parameter without + dnl some very ugly code changes. + if test "$xemacs_compiler" = "g++"; then xe_cflags_warning="$cflags_warning -Weffc++" + elif test "$__GCC3" = "yes"; then + xe_cflags_warning="$cflags_warning -Wunused-parameter" + fi cflags_warning="$cflags_warning -Wshadow -Wmissing-declarations" dnl **** If more gcc/g++ flags are added, from here on must handle dnl **** cflags_warning and xe_cflags_warning in parallel