cvs commit: src/sys/dev/led led.h
Maxime Henrion
mux at freebsd.org
Mon May 10 06:13:13 PDT 2004
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
More information about the cvs-src
mailing list