Re: svn commit: r370938 [1/50] - in /struts: action/trunk/ action/trunk/conf/java/ action/trunk/src/java/org/apache/struts/ action/trunk/src/java/org/apache/struts/action/ action/trunk/src/java/org/ap

2006-01-22 Thread Ted Husted
On 1/22/06, Martin Cooper <[EMAIL PROTECTED]> wrote: > > When the macro expands, it can exceed 80 characters per line, which > > conflicts with the CheckStyle settings. To use the SVN macro, we > > should increase the standard line length to at least 85 characters. > > That's not necessary. Checkst

Re: svn commit: r370938 [1/50] - in /struts: action/trunk/ action/trunk/conf/java/ action/trunk/src/java/org/apache/struts/ action/trunk/src/java/org/apache/struts/action/ action/trunk/src/java/org/ap

2006-01-22 Thread Martin Cooper
On 1/22/06, Ted Husted <[EMAIL PROTECTED]> wrote: > > On 1/20/06, Niall Pemberton <[EMAIL PROTECTED]> wrote: > > I haven't time to look through all 50 commit messages for this change - > but > > a quick look at a few of the messages already brought up a couple of > issues: > > > > * The reformattin

Re: svn commit: r370938 [1/50] - in /struts: action/trunk/ action/trunk/conf/java/ action/trunk/src/java/org/apache/struts/ action/trunk/src/java/org/apache/struts/action/ action/trunk/src/java/org/ap

2006-01-22 Thread Ted Husted
On 1/20/06, Niall Pemberton <[EMAIL PROTECTED]> wrote: > I haven't time to look through all 50 commit messages for this change - but > a quick look at a few of the messages already brought up a couple of issues: > > * The reformatting of the svn keyswords @version $Rev$ $Date$ has moved the > $ sig

Re: svn commit: r370938 [1/50] - in /struts: action/trunk/ action/trunk/conf/java/ action/trunk/src/java/org/apache/struts/ action/trunk/src/java/org/apache/struts/action/ action/trunk/src/java/org/ap

2006-01-20 Thread Niall Pemberton
I haven't time to look through all 50 commit messages for this change - but a quick look at a few of the messages already brought up a couple of issues: * The reformatting of the svn keyswords @version $Rev$ $Date$ has moved the $ sign from the end of Date to the next line - effectively disabling