Mail Archives: geda-user/2016/01/17/16:47:38
> A number of devels don't actually like the current preferred style, but
> want to continue using it perhaps out of respect for the original devels.
> Who aren't even around any more.
No, the request is that you continue to honor whatever style was there
before, so that the style remains consistent throughout, which
increases readability.
Arbitrary changes to style cause diffs to be much bigger than are
needed to convey the meaning of a change, so you should never
gratuitously change the style of code in the same patch as anything
else. Requests to do nothing other than change the style result in a
history that hides other changes behind the style change.
So stick with the existing style, and don't change it unless you have
a REALLY good reason to.
> Indentation matters, local consistency matters, // vs /* and which line
> braces go on don't.
// vs /* matters if the compiler doesn't support C99 yet.
Granted, most do, but nobody's actually clarified that issue. Until
then, and until we officially require a C99 compiler *and test for
it*, // is still to be avoided.
- Raw text -