Ooops

[Kevin O'Neill]

Looks like I've been a bad boy with the coding conventions :(. This
stuff borders on religious arguments so I'm happy to go with the flow.

The _ before a field is a very old habit that has saved my ass many
times and makes seeing the context of the variables (for me) easier. I
also expect that you guys don't use tabs right (yeah looks like 4
spaces).

Sorry.

-k.

PS: are the coding conventions written anywhere :(

[bob mcwhirter]

Post by Kevin O'Neill
Looks like I've been a bad boy with the coding conventions :(. This
stuff borders on religious arguments so I'm happy to go with the flow.

Heh. No comment. :)

Post by Kevin O'Neill
The _ before a field is a very old habit that has saved my ass many
times and makes seeing the context of the variables (for me) easier. I
also expect that you guys don't use tabs right (yeah looks like 4
spaces).

Yah, I did that for a while after migrating from after_ in C++.

Now, I prefer to do this.field, and typically only once within
an accessor that is used everywhere else.

Spaces, yah, 4. Curlies on newlines. Spaces around call params,
break param lists across lines, etc.

Post by Kevin O'Neill
PS: are the coding conventions written anywhere :(

Only by example and as enforced by checkstyle.

'maven site' will produce a checkstyle report.

Also, you might want to turn off those auto-gen'd javadocs
that provide no information. If you leave'em blank, checkstyle
will catch it. If you have default "fill in details here"
javadocs, then checkstyle doesn't know it's bogus.

-bob