From owner-freebsd-current Sun Sep 10 03:45:56 1995 Return-Path: current-owner Received: (from majordom@localhost) by freefall.freebsd.org (8.6.12/8.6.6) id DAA14627 for current-outgoing; Sun, 10 Sep 1995 03:45:56 -0700 Received: from irz301.inf.tu-dresden.de (irz301.inf.tu-dresden.de [141.76.1.11]) by freefall.freebsd.org (8.6.12/8.6.6) with ESMTP id DAA14613 for ; Sun, 10 Sep 1995 03:45:53 -0700 Received: from sax.sax.de by irz301.inf.tu-dresden.de (8.6.12/8.6.12-s1) with ESMTP id MAA17909 for ; Sun, 10 Sep 1995 12:37:23 +0200 Received: by sax.sax.de (8.6.11/8.6.12-s1) with UUCP id MAA18606 for current@FreeBSD.ORG; Sun, 10 Sep 1995 12:37:22 +0200 Received: (from j@localhost) by uriah.heep.sax.de (8.6.12/8.6.9) id JAA20649 for current@FreeBSD.ORG; Sun, 10 Sep 1995 09:14:42 +0200 From: J Wunsch Message-Id: <199509100714.JAA20649@uriah.heep.sax.de> Subject: Re: test coverage of LINT To: current@FreeBSD.ORG Date: Sun, 10 Sep 1995 09:14:40 +0200 (MET DST) Reply-To: current@FreeBSD.ORG In-Reply-To: <199509100344.NAA19536@godzilla.zeta.org.au> from "Bruce Evans" at Sep 10, 95 01:44:35 pm X-Phone: +49-351-2012 669 X-Mailer: ELM [version 2.4 PL23] MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit Content-Length: 1129 Sender: current-owner@FreeBSD.ORG Precedence: bulk As Bruce Evans wrote: > > `grep 'define N.*0'' in /sys/compile/LINT should produce no output but > actually produces the left half of the following: > od.h:#define NOD 0 # source files missing Under development. Only the hooks are reserved yet, there used to be some discussion on the SCSI list about required changes to the SCSI configuration stuff. > su.h:#define NSU 0 # should be enabled ??? IMHO, it's obsolete and has been replaced by the `uk' device. Julian? Peter? > BTW1, most drivers are unecessarily ifdefed with > > #include "foo.h" > #if NFOO > 0 > > #endif > > NFOO is guaranteed to be > 0 if the driver is configured (except possibly > for cases involving cross references). I'd like to remove these ifdefs. I'd rather temporarily add an #else\n#error "Should not happen" to see if there are some weird constellations. You won't recognize any ill side-effects (except of the bloat) otherwise (the driver won't be referenced anyway). -- cheers, J"org joerg_wunsch@uriah.heep.sax.de -- http://www.sax.de/~joerg/ Never trust an operating system you don't have sources for. ;-)