Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 22 Oct 2012 09:31:09 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        Warner Losh <imp@bsdimp.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Warner Losh <imp@freebsd.org>
Subject:   Re: svn commit: r241757 - head/usr.sbin/pciconf
Message-ID:  <201210220931.09563.jhb@freebsd.org>
In-Reply-To: <12563773-B5F9-4F48-A1A7-45E15296F9C4@bsdimp.com>
References:  <201210192248.q9JMmMpt093234@svn.freebsd.org> <201210200830.26755.jhb@freebsd.org> <12563773-B5F9-4F48-A1A7-45E15296F9C4@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sunday, October 21, 2012 11:11:09 pm Warner Losh wrote:
> 
> On Oct 20, 2012, at 6:30 AM, John Baldwin wrote:
> 
> > On Friday, October 19, 2012 06:48:22 PM Warner Losh wrote:
> >> Author: imp
> >> Date: Fri Oct 19 22:48:22 2012
> >> New Revision: 241757
> >> URL: http://svn.freebsd.org/changeset/base/241757
> >> 
> >> Log:
> >>  Indent ecaps the same way we indent caps.
> > 
> > This was on purpose so the '=' signs line up
> 
> It makes it hard to parse otherwise...  Perhaps the equal lining up should 
be documented? Or at least be fixed to match everything else.  When ecaps were 
added the broke indentation...
> 
> I'm happy to fix the alignment issue.

Alternatively you would have to reindent everything else to allow for the
larger 'ecap XXXX[XXX]' width.  You could do that if you wanted to instead.

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201210220931.09563.jhb>