Date: Wed, 4 Feb 1998 22:06:15 -0500 (EST) From: Garrett Wollman <wollman@khavrinen.lcs.mit.edu> To: Eivind Eklund <eivind@yes.no> Cc: cvs-committers@FreeBSD.ORG Subject: Re: cvs commit: src/sys/conf options src/sys/gnu/ext2fs ext2_alloc.c ext2_balloc.c ext2_inode.c ext2_lookup.c ext2_subr.c ext2_vnops.c src/sys/i386/i386 busdma_machdep.c machdep.c pmap.c swtch.s trap.c vm86.c vm_machdep.c src/sys/i386/ibcs2 ibcs2_sysi86. Message-ID: <199802050306.WAA19161@khavrinen.lcs.mit.edu> In-Reply-To: <19980205035631.28915@follo.net> References: <199802042234.OAA23694@freefall.freebsd.org> <199802050239.VAA19082@khavrinen.lcs.mit.edu> <19980205035631.28915@follo.net>
next in thread | previous in thread | raw e-mail | index | archive | help
<<On Thu, 5 Feb 1998 03:56:31 +0100, Eivind Eklund <eivind@yes.no> said: > However, there are some uses of DIAGNOSTIC (e.g, in vm_zone.h) that require > it to be enabled for corresponding C-files to get support routines. This > implies that at least some people use it as a global option. Perhaps the > correct thing to do would be to split the option? Well, the way I've always used it was to turn it on for just those specific modules I was debugging. That keeps the code perturbation down to a minimum. I think it's a safe bet that anybody who really needs to use this option knows what he is doing and can deal with the necessary choices involved. -GAWollman -- Garrett A. Wollman | O Siem / We are all family / O Siem / We're all the same wollman@lcs.mit.edu | O Siem / The fires of freedom Opinions not those of| Dance in the burning flame MIT, LCS, CRS, or NSA| - Susan Aglukark and Chad Irschick
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199802050306.WAA19161>