Date: Mon, 10 May 2004 15:13:13 +0200 From: Maxime Henrion <mux@freebsd.org> To: Eivind Eklund <eivind@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/led led.h Message-ID: <20040510131313.GG700@elvis.mu.org> In-Reply-To: <20040510130612.GA14041@FreeBSD.org> References: <200405101150.i4ABoL9V065750@repoman.freebsd.org> <20040510115040.2BD4B16A53F@hub.freebsd.org> <20040510130612.GA14041@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Eivind Eklund wrote: > Not a big deal, but a minor style issue: > > On Mon, May 10, 2004 at 04:50:40AM -0700, Maxime Henrion wrote: > > - Fix some style bugs while I'm here. > > [...] > > > -#endif /* _DEV_LED_H */ > > +#endif > > Based in my presently installed /usr/include/sys, the style statistics > for /usr/include/sys/*.h is as follows: > > #endif #endif /* _DEV_LED_H */ #endif /* !_DEV_LED_H */ > 28 52 148 > 12% 23% 65% > > The removal of /* _DEV_LED_H */ from the last #endif brings it from an > uncommon style to an even less common style. Both styles are allowed by style(9) under different circumstances. If the #ifdef'ed block is big, then you should add the /* FOO_H */ comment. If it's not, you shouldn't put it, hence why I removed it. So this isn't a matter of which style is used most :-). Cheers, Maxime
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040510131313.GG700>