[jira] Resolved: (STR-2900) Checkstyle patches

["Wendy Smoak (JIRA)" <ji...@apache.org>]

     [ http://issues.apache.org/struts/browse/STR-2900?page=all ]
     
Wendy Smoak resolved STR-2900:
------------------------------

    Resolution: Fixed
     Assign To: Wendy Smoak  (was: James Mitchell)

Done.  Thanks again, Frank!

I skipped the two Faces example apps.  They were mostly tabs -> spaces, and the patch only used two spaces per tab rather than four, so the indenting was wrong.


> Checkstyle patches
> ------------------
>
>          Key: STR-2900
>          URL: http://issues.apache.org/struts/browse/STR-2900
>      Project: Struts 1
>         Type: Improvement

>   Components: Core, Apps, EL, Extras, Faces, Scripting, Taglibs, Tiles
>     Versions: 1.3.1
>  Environment: Any
>     Reporter: Frank W. Zammetti
>     Assignee: Wendy Smoak
>      Fix For: 1.3.5
>  Attachments: all.patch, apps.zip, core.zip, el.zip, extras.zip, faces.zip, frank_checkstyle.patch.gz, integration.zip, mailreader-dao.zip, scripting.zip, taglib.zip, tiles.zip
>
> This is a collection of patches which addresses about 2,150 Checkstyle complaints.  Each ZIP file should be unzipped, and be sure to recreate directory structure!  You will find a structure that matches the source repo.  I created a single patch file for each package within each subproject that has changes, so it should be quite granular enough.  Also note that there are some new package.html files in some places which should be added.  I will be happy to do another round once these are applied.
> For background, see also 
> * http://www.mail-archive.com/dev%40struts.apache.org/msg22827.html
> * http://www.mail-archive.com/dev%40struts.apache.org/msg22988.html

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira