The "sun_checks.xml" provided by Checkstyle actually goes further than
the Sun Coding Conventions. It also has rules from the Java Language
Specification and some general best practices. I pointed this out to the
Checkstyle team earlier and suggested that they would separate these.
Mick Knutson wrote:
I just got maven jalopy working, and I got a jalopy.xml of the standard sun
standards.
I formatted all my code but then checkstyle went from 40 errors to >700
errors.
Many of which are related to braces.
'(' is followed by
whitespace.238<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#238>[image:
error]')' is preceded with
whitespace.238<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#238>[image:
error]'if' construct must use
'{}'s.240<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#240>[image:
error]'(' is followed by
whitespace.240<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#240>[image:
error]')' is preceded with
whitespace.240<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#240>[image:
error]'if' construct must use
'{}'s.242<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#242>[image:
error]'(' is followed by
whitespace.242<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#242>[image:
error]')' is preceded with
whitespace.242<file:///C:/temp/site-1.0.0.0/utilities/common/common-jar/xref/com/baselogic/util/xml/XMLParser.html#242>[image:
error]'if' construct must use '{}'s.
So, what should I do? Should I ask for help to get a checkstyle.xml
template
that is more aligned with my jalopy.xml.
Or should I modify my jalopy.xml to use the brace specification that
checkstyle is complaining about?
--
Dennis Lundberg
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
