[ http://issues.apache.org/struts/browse/STR-2889?page=comments#action_37569 ]
James Mitchell commented on STR-2889: ------------------------------------- Frank, thanks for the patch. By the way, can you turn off 'remove line endings' from your editor? As you probably noticed, it just adds noise to the signal. Nonetheless, thanks again. -- James Mitchell > Updated Checkstyle rules > ------------------------ > > Key: STR-2889 > URL: http://issues.apache.org/struts/browse/STR-2889 > Project: Struts Action 1 > Type: Improvement > Components: Core, Apps, EL, Extras, Faces, Scripting, Taglibs, Tiles > Versions: 1.2 Family, 1.3 Family > Environment: Any > Reporter: Frank W. Zammetti > Attachments: checkstyle_rules.patch > > 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/[EMAIL > PROTECTED] > 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. -- 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
