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]

Reply via email to