From: djm <djm@openbsd.org>
Date: Fri, 2 Jun 2017 06:06:10 +0000 (+0000)
Subject: rationalise the long list of manual CDIAGFLAGS that we add; most of
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=84e9c7c06d59fb0d37d73588beb54a36abab46a4;p=openbsd

rationalise the long list of manual CDIAGFLAGS that we add; most of
these were redundant to -Wall -Wextra
---

diff --git a/usr.bin/ssh/Makefile.inc b/usr.bin/ssh/Makefile.inc
index 1d9529fff89..4962ae662c4 100644
--- a/usr.bin/ssh/Makefile.inc
+++ b/usr.bin/ssh/Makefile.inc
@@ -1,4 +1,4 @@
-#	$OpenBSD: Makefile.inc,v 1.54 2017/04/30 23:25:03 djm Exp $
+#	$OpenBSD: Makefile.inc,v 1.55 2017/06/02 06:06:10 djm Exp $
 
 .include <bsd.own.mk>
 
@@ -6,36 +6,23 @@ CFLAGS+=	-I${.CURDIR}/..
 CFLAGS+=	-fstack-protector-all
 
 CDIAGFLAGS=	-Wall
-CDIAGFLAGS+=	-Wchar-subscripts
-CDIAGFLAGS+=	-Wcomment
 CDIAGFLAGS+=	-Wextra
-CDIAGFLAGS+=	-Wformat
 CDIAGFLAGS+=	-Wformat-security
-CDIAGFLAGS+=	-Wimplicit
-CDIAGFLAGS+=	-Winline
 CDIAGFLAGS+=	-Wmissing-declarations
 CDIAGFLAGS+=	-Wmissing-prototypes
-CDIAGFLAGS+=	-Wparentheses
 CDIAGFLAGS+=	-Wpointer-arith
-CDIAGFLAGS+=	-Wreturn-type
 CDIAGFLAGS+=	-Wshadow
-CDIAGFLAGS+=	-Wsign-compare
-CDIAGFLAGS+=	-Wstrict-aliasing
 CDIAGFLAGS+=	-Wstrict-prototypes
-CDIAGFLAGS+=	-Wswitch
-CDIAGFLAGS+=	-Wtrigraphs
-CDIAGFLAGS+=	-Wuninitialized
 CDIAGFLAGS+=	-Wunused
+CDIAGFLAGS+=	-Wno-unused-parameter # Lots of these in protocol handlers.
 .if ${COMPILER_VERSION:L} != "gcc3"
+CDIAGFLAGS+=	-Wstrict-aliasing=2
 CDIAGFLAGS+=	-Wold-style-definition
 .endif
-CDIAGFLAGS+=	-Wno-unused-parameter # Not clean for this yet
-#CDIAGFLAGS+=	-Wwrite-strings # XXX too messy; needs libc changes
-#CDIAGFLAGS+=	-Wcast-qual # XXX not clean
-#CDIAGFLAGS+=	-Wcast-align # XXX can't be clean with sockaddr casts?
 
 #CDIAGFLAGS+=	-Werror
 #DEBUG=-g
+#INSTALL_STRIP=
 
 WARNINGS=yes