Excellent work. I have a problem with three rules: major: The "DesignForExtensionCheck" seems a bit silly. minor: The "LeftCurlyCheck" is against my in-house rule that if there is a multi line condition, the { should be on a separate line - but I can live the rule. minor: "RedundantThrowsCheck: It complains about BuildException.
Peter On Thu, 2003-07-03 at 15:26, Conor MacNeill wrote: > I've done a checkstyle audit with the new xdoc generating stylesheets and > uploaded this to the server > > http://ant.apache.org/reports/checkstyle/ > > It's not linked into the rest of the site (nor is it in CVS). It's mainly > just > for interest at the moment. It does show which files are good candidates for > some cleanup action :-). > > BTW, this is checkstyle on maximum volume. I'll probably turn it down a bit a > little later on. It is too late to fix some of our code now, for backward > compatibility reasons, so I'll be trying to focus on what can be fixed. > > If you hover over a line in the detail report it will show which checkstyle > module triggered the report. > > Conor > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]