On Tue, Mar 25, 2008 at 05:07:35PM +0100, Jörn Engel wrote:
It really gains us nothing, however that is not really the point.
The point is that consistancy is good, with the space is the more normal
'C' usage, without for 'C++'; something to do with the implication that
(foo *) is a pointer to a foo (separate things), and (foo*) is a thing
of type pointer to foo (one thing) which is more object oriented.
The "norm" is with and so it makes sense to maintain it that way. A lot
of the layout and style choises are arbitrary, and disliked by many of us,
but we follow the style to maintain that common feel.
Yes that is a false positive. I'll have a look at fixing it.
"else if" is at least creating an additional arm of the same control
structure. else while is mixing two different paradigms.
Because to add the space would mean breaking the line to avoid exceeding
80 characters?
That is a false positive triggered by the ' |S' at the start of
the line. This one is fixed in the head of my tree. The more normal
form would be for that leading | to be on the end of the previous line,
and the exception for that was already there.
-apw
--
