Checkstyle template -- feedback wanted

[David Jencks]

After actually looking at the Checkstyle report for the connector 
module, I noticed that it was complaining about several things the 
Geronimo team and I in particular seem to regard as good coding 
practices.  Therefore I set up a customized geronimo_checks.xml in etc 
and pointed to it in global.properties.

My changes consist of:
--allowing lines up to 120 characters.
--allowing "hiding variables" in constructor arguments and setters.
--disabling the "design for extension" check.
The reference for what can be checked is here:
http://checkstyle.sourceforge.net/config.html#treewalkerchecks
I'd appreciate any comments/complaints/suggestions.
Another check we might want to consider the impact of is 
FinalLocalVariable
http://checkstyle.sourceforge.net/config_coding.html#FinalLocalVariable

This appeared to be on in the default template, although I don't see 
the configuration in my copy.

Thanks,
David Jencks