DO NOT REPLY [Bug 35956] New: - Updated CheckStyle rules file

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35956>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35956

           Summary: Updated CheckStyle rules file
           Product: Struts
           Version: 1.3.0
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P3
         Component: Unknown
        AssignedTo: dev@struts.apache.org
        ReportedBy: fzammetti@omnytex.com


I wanted to contribute to the release of 1.3, and one way I thought I could do 
that would be to tackle some of the CheckStyle errors found in the current 
code (I have something of a fetish for a clean CheckStyle results before a 
release of my own stuff, or at least as close as I can get to totally 
clean!).  I seemed to have some trouble running CheckStyle using the 
CheckStyle.xml rules file I pulled out of SVN, and I suspect it was because I 
am using a newer version of CheckStyle (3.5).

So, attached here is an updated rules file that works under 3.5.

I have also put in a bunch of commented-out modules that I believe would be a 
good idea to be run against the code as well.  If there are enough +1's on 
them, I'd love to see the new file committed with those modules activated... 
I'd be more than happy to attach a version with them all active, but I kinda 
figure anyone could handle that! :)

I hope my intention of dealing with some of the CheckStyle problems is indeed 
a good idea.  You should see at least a few patches tommorrow from me for at 
least some of the more minor issues.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org