summaryrefslogtreecommitdiff
path: root/cflags.SH
diff options
context:
space:
mode:
authorJarkko Hietaniemi <jhi@iki.fi>2014-06-18 08:43:37 -0400
committerJarkko Hietaniemi <jhi@iki.fi>2014-06-18 14:14:27 -0400
commit6a8c321707dde176ebe36b755a02e0561ba16628 (patch)
tree2fcdb698ff71d83034a8692987a0ea3d70970720 /cflags.SH
parentffb6256b1100a793924bee58c67d32d3e2643b59 (diff)
downloadperl-6a8c321707dde176ebe36b755a02e0561ba16628.tar.gz
Make also cflags.SH clean with g++ -Wunused-*
"clean" as in "not needed anymore".
Diffstat (limited to 'cflags.SH')
-rwxr-xr-xcflags.SH33
1 files changed, 0 insertions, 33 deletions
diff --git a/cflags.SH b/cflags.SH
index 5538e92b6d..69ad9995a9 100755
--- a/cflags.SH
+++ b/cflags.SH
@@ -241,39 +241,6 @@ do
esac
done
-# -Wall includes -Wunused-value and -Wunused-parameter, which may be too much
-# with some compilers.
-#
-# XXX this is related to the unfortunate fact that bare -Wall (without
-# amending -Wno-unused-...) is too much for XS code because of all the
-# often generated but unused things.
-#
-case "$ccflags$warn" in
-*-Wall*)
- case "$cc" in
- *g++*)
- # Without -Wno-unused-variable g++ 4.x compiles are rather unwatchable
- # because of all the warnings about Perl___notused, and g++ doesn't do
- # __attribute__((unused)) (and even if at some stage it may, people do
- # have older gcc installations), and ((void)x) isn't enough to silence
- # the noises about XS functions not using their cv parameter, so we need
- # the -Wno-unused-parameter too.
- # Yes, we lose some valid warnings, but hopefully other compilers
- # (like gcc) will still pick up those warnings.
- for f in -Wno-unused-variable -Wno-unused-parameter
- do
- case "$warn" in
- *"$f"*) ;;
- *)
- echo "cflags.SH: Adding $f because of g++."
- warn="$warn $f" ;;
- esac
- done
- ;;
- esac
- ;;
-esac
-
# Code to set any extra flags here.
extra=''