Date: Thu, 31 May 2012 20:04:17 -0700 From: Doug Barton <dougb@FreeBSD.org> To: Eitan Adler <eadler@freebsd.org> Cc: svn-src-head@freebsd.org, Tom Judge <tj@freebsd.org>, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r236362 - head/share/misc Message-ID: <4FC83131.4010702@FreeBSD.org> In-Reply-To: <CAF6rxgmxq-VBKTXNPVPR8czaoQAFKYaOG9ETYN4zyneiefcg0g@mail.gmail.com> References: <201205311933.q4VJXpnE060290@svn.freebsd.org> <4FC81AE3.10304@FreeBSD.org> <CAF6rxgmxq-VBKTXNPVPR8czaoQAFKYaOG9ETYN4zyneiefcg0g@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 5/31/2012 6:59 PM, Eitan Adler wrote:
> On 31 May 2012 18:29, Doug Barton <dougb@freebsd.org> 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 none of the changes were really substantive so it was okay
> to sort and fix the \n at the same time.
These were the changes:
Log:
- Fix my botched merge and correct mentor ordering. (Spotted by flo)
- Fix missing new line in node block. (Spotted by gavin)
- Sort the node list while I'm here.
The last is definitely WS, the second probably is, the first definitely
isn't.
hth,
Doug
--
This .signature sanitized for your protection
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4FC83131.4010702>