Hi Michelle,
checkstyle is a development tool checking whether the code follows a
coding standard. The xml file is used to configure the rules defining
the coding standard. checkstyle is called by the maven plugin 'site'
when generating the site for a project and includes the checkstyle
report in the documentation. So I think the tool is more useful for TCK
developers rather than TCK users.
The configuration sun_checks.xml (the base for the jdo_checks.xml file)
configures checkstyle to check the Sun Code Conventions. The tools is
very picky, so if you run it against the api2 or tck2 sources, it will
find a lot of issues, where you could argue whether this is really bad
code. I relaxed some of the rules (e.g. a simple rule that a line should
have at most 80 characters, I configured it to 100). From my point of
view finishing the checkstyle configuration would mean: agree to a
configuration we want to use in the JDO API and TCK and then change the
sources such that they adhere to it. This seems to be a lot of work and
I don't know whether it is worth doing it.
Regards Michael
Thanks, Michael, I see it there now. I'm afraid, though, that I don't
understand what it does or why the TCK user needs to be aware that the
file is there. Also, what is unfinished of the checkstyle
configuration? Should we finish it?
Thanks,
Michelle
Michael Bouschen wrote:
Hi Michelle, hi Craig,
maybe I can help :-).
You find the file jdo_checks.xml in the repositiry in the top-level
directory, so parallel to api2, tck2, NOTICE.txt, README.txt etc. It
is used when calling 'maven site' which runs checkstyle. I propose to
keep the paragraph in the README.
Regards Michael
No problem by me.
Craig
On Jun 20, 2008, at 5:02 PM, Michelle Caisse wrote:
The README.txt file in trunk/ says:
The file jdo_check.xml includes the checkstyle configuration. It
is borrowed
from the sun_checks.xml, but does not use all of the sun rules and
customizes
some other rules. The checkstyle configuration is not yet finished.
I don't see this file anywhere. In fact, I don't remember ever
seeing it. Can we delete this paragraph?
-- Michelle
Craig Russell
Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!
--
*Michael Bouschen*
*Prokurist*
akquinet [EMAIL PROTECTED] GmbH
Tempelhofer Ufer 23-24, D-10963 Berlin
Bülowstr. 66, D-10783 Berlin
Fon: +49 30 235 520-33
Fax: +49 30 217 520-12
Email: [EMAIL PROTECTED]
Url: www.akquinet.de <http://www.akquinet.de>
akquinet [EMAIL PROTECTED] GmbH, Berlin
Geschäftsführung: Prof. Dr. Christian Roth, Hendrik Saly, Martin Weber
Amtsgericht Berlin-Charlottenburg HRB 86780
USt.-Id. Nr.: DE 225 964 680
