Date: Sun, 22 Jul 2001 01:50:34 +0100 From: Paul Richards <paul@freebsd-services.co.uk> To: Bruce Evans <bde@zeta.org.au>, Ruslan Ermilov <ru@FreeBSD.org> Cc: Bruce Evans <bde@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/share/man/man9 style.9 Message-ID: <517230000.995763034@lobster.originative.co.uk> In-Reply-To: <Pine.BSF.4.21.0107210133070.10938-100000@besplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--On Saturday, July 21, 2001 01:38:06 +1000 Bruce Evans <bde@zeta.org.au>
wrote:
> On Fri, 20 Jul 2001, Ruslan Ermilov wrote:
>
>> On Fri, Jul 20, 2001 at 09:35:41PM +1000, Bruce Evans wrote:
>> [...]
>> > Anyway, the continuation indent rule is at the lexical level, so it
>> > doesn't apply to `else if'.
>> >
>> Doh, and why I have removed that final `else' part? :-)
>>
>> What I was talking about was:
>> if (mylevel == SLC_DEFAULT) {
>> slctab[(int)func].current.flag = flag;
>> slctab[(int)func].current.val = val;
>> flag |= SLC_ACK;
>> } else if (hislevel == SLC_CANTCHANGE &&
>> mylevel == SLC_CANTCHANGE) {
Why is this good style?
I find that much harder to read than the case where the tests are lined up
i.e.
} else if (hislevel == SLC_CANTCHANGE &&
mylevel == SLC_CANTCHANGE) {
Paul.
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?517230000.995763034>