i make it a habit to always look over the svn diff output before committing anything. this makes it pretty hard to miss unintended formatting changes.
On 9/27/05, Will Glass-Husain <[EMAIL PROTECTED]> wrote: > Hi Daniel, > > My IDE is set to convert tabs to 4 spaces. Maybe that's the issue. I'll > keep an eye on this. > > WILL > > ----- Original Message ----- > From: "Daniel L. Rall" <[email protected]> > To: "Velocity Developers List" <[email protected]> > Sent: Tuesday, September 27, 2005 12:02 PM > Subject: non-functional code changes > > > > On Tue, 27 Sep 2005, Henning Schmiedehausen wrote: > > > >> Ok, so how about this (I admit that I'm guilty of removing trailing > >> blanks from some files that I work on and sneak these changes in): > > > > You're not the only one. I abhor trailing whitespace, so I constantly > > feel > > your compulsion... ;) > > > > Will, is your editor configured to do this automatically? Many changes > > you've checked in recently have been obscured by numerous whitespace > > tweaks. > > > >> I keep my fingers still until we have all the code changes in and call > >> it RC1. Afterwards we do one big rush to clean up these issues like > >> trailing blanks and missing javadocs. So we get all these cosmetic > >> changes isolated in one go. > > > > You guys are pushing this release, so I'll leave it to you. However, my > > personal preference is that you do the changes pre-RC (like you've been > > doing, but in one or two fell sweeps); once an RC has been cut, the only > > code changes it should _ever_ see are medium to severe defect fixes, or > > necessary API adjustments/additions. > > > > - Dan > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
