svn commit: r186751 - in head/sys: netinet6 netipsec

M. Warner Losh imp at bsdimp.com
Tue Jan 6 12:57:02 PST 2009


In message: <49627592.60208 at gmx.de>
            Christoph Mallon <christoph.mallon at gmx.de> writes:
: Steve Kargl schrieb:
: >     Stylistic changes (including whitespace changes) are hard on
: > 	the source repository and are to be avoided without good reason. 
: 
: Yet another anachronism in style(9).
: (Does that sentence even mean what I think it means? Somehow it sounds 
: like "there is no good reason, but avoid them anyway" instead of "avoid 
: them if you have no good reason".)

It means "Don't make white space changes because you can, make them
because you have some other reason for changing the code."  If you are
just reading something and notice a space out of whack, don't bother.
If you are debugging the code, fixing a bug, adding a feature, go
ahead and fix that too: but make two commits.

Warner



More information about the svn-src-head mailing list