From owner-svn-src-all@FreeBSD.ORG Wed Dec 30 07:09:47 2009 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 956CF1065670; Wed, 30 Dec 2009 07:09:47 +0000 (UTC) (envelope-from delphij@gmail.com) Received: from mail-pw0-f44.google.com (mail-pw0-f44.google.com [209.85.160.44]) by mx1.freebsd.org (Postfix) with ESMTP id 536F28FC1D; Wed, 30 Dec 2009 07:09:47 +0000 (UTC) Received: by pwi15 with SMTP id 15so7917727pwi.3 for ; Tue, 29 Dec 2009 23:09:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=LCAUDvwWGluhc+acmg5AYrYAUUzakvAbtfn2L9DqF9U=; b=p4r4hJ2GGPdXE8Sr6+SdcrqU1AO3hp1y8HSWGBdOTIXS2xKkTN5DewM/ng22YrxIVX SFxLd/2JPhlRevDMxbx/mkZV0VkOlsGjjD6hnyBiUFe700/v+GRhCF5ULxGccS2Keq+K VANaDqEa/VNjX9iHXKpnzimujObjhgkHc4V0A= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=DU2zxeQ09hHDHYH5NH6EGJQiI4UZwVde2HUy8MvsrX49PhC51Us1BbzUXhQ2Ozx4jS 1MkjtgQbaXwMTlHj9l8XIptu+pj9I/uCpNrBE23OKwOn7GtZzIRa4HyxAeJIni2d7f2N 87Yot62+e4Ntcv2YMRTFn7146UNJv41ZYGsm0= MIME-Version: 1.0 Received: by 10.115.117.9 with SMTP id u9mr12185617wam.172.1262156977391; Tue, 29 Dec 2009 23:09:37 -0800 (PST) In-Reply-To: <20091230064030.GK64905@hoeg.nl> References: <200912292253.nBTMrR5m038869@svn.freebsd.org> <4B3AA7DC.1040606@FreeBSD.org> <20091230062236.GI64905@hoeg.nl> <20091230064030.GK64905@hoeg.nl> Date: Tue, 29 Dec 2009 23:09:37 -0800 Message-ID: From: Xin LI To: Ed Schouten Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Gabor Kovesdan Subject: Re: svn commit: r201227 - in head: sbin/comcontrol sbin/mount sbin/mount_msdosfs sbin/mount_nullfs sbin/rcorder usr.bin/find usr.bin/gencat usr.bin/mklocale usr.sbin/config usr.sbin/cpucontrol usr.sbin... X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 30 Dec 2009 07:09:47 -0000 On Tue, Dec 29, 2009 at 10:40 PM, Ed Schouten wrote: > * Xin LI wrote: >> On Tue, Dec 29, 2009 at 10:22 PM, Ed Schouten wrote: >> > Hi Gabor, >> > >> > * Gabor Kovesdan wrote: >> >> I wonder if it would be worth to invent a WARNS=3D7 level, which woul= d >> >> be WARNS=3D6 + ANSI. Although ANSI-conformance isn't a -Wfoo flag, >> >> which is WARNS for but conforming ANSI means a higher level >> >> cleannes, which is demanded, so I think it wouldn't really >> >> contradict to the semantics of WARNS. >> > >> > So far I've been using -Wold-style-definition, which I've been using >> > make these changes. I am planning on just adding that to WARNS=3D6, >> > considering that almost all the code at WARNS=3D6 builds with this fla= g >> > enabled now. >> > >> > I'd rather not add a WARNS=3D7, because then I'm afraid almost nobody = will >> > know about its existence, causing it to be mostly ignored. In an ideal >> > world, I think we should get rid of WARNS. We could just enable all -W >> > flags by default, except -Werror. That way people would be invited to >> > fix the warnings over time. >> >> Probably leaving -Werror by default is better... =C2=A0Without -Werror I >> think there is fewer chance that these issues being noticed. =C2=A0The >> point of having -Werror is that it means a build break (at the >> developers' local machine, or tinderbox if he or she didn't do a full >> universe build), loud enough and can be fixed early rather than >> letting them go. =C2=A0Historically we only disable -Werror when major >> compiler change, etc. > > Sure. We should just enable it in Makefile.inc by default, but basically > my proposal is to just reduce the entire warnings thing to just two > options: > > - Have all warnings enabled, but not -Werror. > - Have all warnings enabled and also -Werror. > > Where we enable the latter is another discussion. Oh... There are a lot of contributed code that can not even pass WARNS=3D1 :) Also there are some code "for historical reasons" must cast off qualifiers, etc., currently our build system have warning levels assigned at module level, which could be inconvenient for this policy to be effective (e.g. a 100-files module with 95 files WARNS=3D6 clean and 5 rest are only WARNS=3D4 clean, we can use WARNS=3D4 and WERROR=3D1 to make sure that no file would get worse, but with WARNS=3D=3D= =3D6 and WERROR either 1 or 0, we could lose the ability to limit the bit rot, except we convert all these 5 files as a wholesale :) > (Almost happy new year), Happy new year! Cheers, --=20 Xin LI http://www.delphij.net