From owner-cvs-all  Fri Jul 20  9:24: 1 2001
Delivered-To: cvs-all@freebsd.org
Received: from rover.village.org (rover.bsdimp.com [204.144.255.66])
	by hub.freebsd.org (Postfix) with ESMTP
	id 48D6137B401; Fri, 20 Jul 2001 09:23:51 -0700 (PDT)
	(envelope-from imp@harmony.village.org)
Received: from harmony.village.org (harmony.village.org [10.0.0.6])
	by rover.village.org (8.11.3/8.11.3) with ESMTP id f6KGNoF38058;
	Fri, 20 Jul 2001 10:23:50 -0600 (MDT)
	(envelope-from imp@harmony.village.org)
Received: from harmony.village.org (localhost.village.org [127.0.0.1])
	by harmony.village.org (8.11.3/8.11.4) with ESMTP id f6KGNno66864;
	Fri, 20 Jul 2001 10:23:49 -0600 (MDT)
	(envelope-from imp@harmony.village.org)
Message-Id: <200107201623.f6KGNno66864@harmony.village.org>
To: Ruslan Ermilov <ru@FreeBSD.org>
Subject: Re: cvs commit: src/share/man/man9 style.9 
Cc: Bruce Evans <bde@FreeBSD.org>, cvs-committers@FreeBSD.org,
	cvs-all@FreeBSD.org
In-reply-to: Your message of "Fri, 20 Jul 2001 11:33:10 +0300."
		<20010720113310.C30828@sunbay.com> 
References: <20010720113310.C30828@sunbay.com>  <200107191803.f6JI3he58920@freefall.freebsd.org> <20010719152744.A37403@sneakerz.org> 
Date: Fri, 20 Jul 2001 10:23:49 -0600
From: Warner Losh <imp@harmony.village.org>
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

In message <20010720113310.C30828@sunbay.com> Ruslan Ermilov writes:
: This is using style(9)'s second-level of four indents rule.
: But, strictly speaking, `else if' is not an operator, this
: is `else' operator followed by the `if' operator.  So if we
: would use the second-level indent of four spaces relatively
: to the `if' operator, we would get:
: 
: 		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) {
: 			flag &= ~SLC_LEVELBITS;
: 			flag |= SLC_NOSUPPORT;
: 			slctab[(int)func].current.flag = flag;
: 		}
: 
: Which is much better, no?  Hell, but then, I guess, we would need
: to indent lines within `else if' block relative to `if' as well.

I don't think it is better.  If nothing else, emacs will have a harder
time doing that because you are indenting 4 spaces most of the time,
and 11 in this case. :-(

Warner

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message