FWIW, these are my preferences

> Coding convention:
> 1. Brace in new lines for class
> Ara: +1
> Mathias: -1
> David: +1
> Konstantin: -1
> Aslak: -1
> Vincent: +1
Andrew: +1

> 2. Brace in new lines for methods
> Ara: +1
> Mathias: -1
> David: +1
> Konstantin: -1
> Aslak: -1
> Vincent: +1
Andrew +1
 
> 3: No @todo-javadoc stubs
> Ara: +1
> Mathias: +1
> David: +0
> Konstantin: -1
> Aslak: +1
> Vincent: +1
Andrew: +1
 
> 4. Spaces before/after parenthesis
> Ara: +1
> Mathias: -1
> David: -1
> Konstantin: +1
> Aslak: +1
> Vincent: -1
Andrew: -1

> Checkstyle:
> 1. Use checkstyle out of the box (sun's convention)
> Mathias: +1
> Vincent: +1
> 
> 2. Use checkstyle with some modifications
> Ara: +1
> David: +1
> Konstantin: + 1
> Aslak: +1

Whichever the above requires.

By the way, does checkstyle actually change the code to these conventions 
(like pretty), or just check that it fits it? (their project web site says 
"It automates the process of checking Java code" which suggests the 
latter, but that's nothing like as useful).

Also, what advantages does it have over pretty, which (apart from the 
occasional EOL strangeness) seems to be doing the job okay up to now?  
Does it understand CVS enough to not reformat if the files aren't 
modified, etc.?

Oh yes, and there's a few more formatting things to consider (don't know 
if checkstyle does them but I'll mention them anyway):

5. methods' throws clause on separate line, indented twice.

6. class'/interfaces' extends/implements on separate line, indented twice.

7. if a statement/expression gets split over >1 line, indent the 
continuation lines twice.

And I prefer 4 spaces, not tabs, for the indents and braces on same line 
for everything other than classes/methods.

> 3. don't use


Andrew.

_______________________________________________________________

Have big pipes? SourceForge.net is looking for download mirrors. We supply
the hardware. You get the recognition. Email Us: [EMAIL PROTECTED]
_______________________________________________
Xdoclet-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/xdoclet-devel

Reply via email to