Date: Tue, 29 Dec 2009 23:09:37 -0800 From: Xin LI <delphij@gmail.com> To: Ed Schouten <ed@80386.nl> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Gabor Kovesdan <gabor@freebsd.org> 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... Message-ID: <a78074950912292309t4f7b5bdfscf03648c05892c4f@mail.gmail.com> In-Reply-To: <20091230064030.GK64905@hoeg.nl> References: <200912292253.nBTMrR5m038869@svn.freebsd.org> <4B3AA7DC.1040606@FreeBSD.org> <20091230062236.GI64905@hoeg.nl> <a78074950912292228w2ca11ba1uc741307bfff49a39@mail.gmail.com> <20091230064030.GK64905@hoeg.nl>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Dec 29, 2009 at 10:40 PM, Ed Schouten <ed@80386.nl> wrote: > * Xin LI <delphij@gmail.com> wrote: >> On Tue, Dec 29, 2009 at 10:22 PM, Ed Schouten <ed@80386.nl> wrote: >> > Hi Gabor, >> > >> > * Gabor Kovesdan <gabor@FreeBSD.org> 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 <delphij@delphij.net> http://www.delphij.net
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?a78074950912292309t4f7b5bdfscf03648c05892c4f>