svn commit: r190098 - in head/sys/sparc64: fhc sparc64

Julian Elischer julian at elischer.org
Sun Mar 22 11:15:51 PDT 2009


Christoph Mallon wrote:
> Andriy Gapon schrieb:
 >> compliants about style (9)
 > agreement and disagreement.


basically almost every single committer who comes into the group
usually starts up by objecting to one part or another of the style
guide. However the problem is that we all object to different parts
and those parts that others disagree with are often parts that
we see as being reasonable.


examples:
" I wan to go > 80 columns".

and

"I often end up debugging on the console. thank goodness the
sources fit in 80 columns".


The style guide IS open to change but not by any individual
contributer.  Most of the things in there are there for a reason that
is non obvious.   Sometimes they are arbitrary.


If one has read BSD code and other system code however, one is 
sometimes struck with how uniform the BSD code is. Once you get
the style into your head you can start taking shortcuts.. For
example you know where all the locals are, and you can see from
an quick look how much stack space a particular function is
contributing to using up the very limited stack space available
in kernel.

At this stage you will find that there are just some things
you will have to live with because changing them now would make new 
sources just too different too old sources.
(I'd put indent rules and 8-char tabs into that category.)
There are other things that will be amenable to change with sufficient
discussion.  Due to different compiler rules for example, there may be
reason to drop some rules because they were there to stop people
from making mistakes that teh compiler now warns about. But generally
every change needs to pass one question.

"Will the code still largely 'smell' like BSD code?"




More information about the svn-src-all mailing list