Re: svn commit: r236362 - head/share/misc

2012-05-31 Thread Doug Barton
On 5/31/2012 6:59 PM, Eitan Adler wrote: > On 31 May 2012 18:29, Doug Barton wrote: >> On 5/31/2012 12:33 PM, Tom Judge wrote: >>> - Sort the node list while I'm here. >> >> This falls under the category of a "whitespace" change, which should >> have been done as a separate commit (usually first

Re: svn commit: r236362 - head/share/misc

2012-05-31 Thread Tom Judge
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 31/05/2012 21:29, Doug Barton wrote: > On 5/31/2012 12:33 PM, Tom Judge wrote: >> - Sort the node list while I'm here. > > This falls under the category of a "whitespace" change, which > should have been done as a separate commit (usually first, th

Re: svn commit: r236362 - head/share/misc

2012-05-31 Thread Eitan Adler
On 31 May 2012 18:29, Doug Barton wrote: > On 5/31/2012 12:33 PM, Tom Judge wrote: >>   - Sort the node list while I'm here. > > This falls under the category of a "whitespace" change, which should > have been done as a separate commit (usually first, then substantive > changes second). I figured

Re: svn commit: r236362 - head/share/misc

2012-05-31 Thread Doug Barton
On 5/31/2012 12:33 PM, Tom Judge wrote: > - Sort the node list while I'm here. This falls under the category of a "whitespace" change, which should have been done as a separate commit (usually first, then substantive changes second). This is a tiny bit more work for you, but down the road when