[STR-2889] Updated Checkstyle rules

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.2 Family, 1.3 Family
• Fix Version/s: 1.3.5
• Component/s: Core, Example Applications, Extras, JSF Integration (Dormant), Scripting, Tag Libraries, Tag Libraries EL (Dormant), Tiles 1 Plugin
• Environment: Any
• Flags: Patch
• Assignee: James Mitchell
• Reporter: Frank W. Zammetti
• Created: 20/Jun/06 12:19
• Updated: 04/Jul/07 03:25
• Resolved: 12/Jul/06 00:45

Description

Argh, I have the priority choices... shouldn't there be something in between minor and major?!? But I digress...

With reference to this thread on the @dev list:

http://mail-archives.apache.org/mod_mbox/struts-dev/200606.mbox/[e-mail]%3e

Attached is a patch for the Checkstyle rules that eliminates the two things discussed in this thread. It resolves a few hundred exceptions in core alone (350-400, give or take), and should do the same for all other subprojects. There seemed to be a feeling (don't want to say consensus at this point!) that applying this before the next milestone would be acceptable.