From owner-cvs-all Fri Jul 20 8:40:40 2001 Delivered-To: cvs-all@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 59D9A37B403; Fri, 20 Jul 2001 08:40:26 -0700 (PDT) (envelope-from bde@zeta.org.au) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id BAA13989; Sat, 21 Jul 2001 01:40:23 +1000 Date: Sat, 21 Jul 2001 01:38:06 +1000 (EST) From: Bruce Evans X-Sender: bde@besplex.bde.org To: Ruslan Ermilov Cc: Bruce Evans , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/share/man/man9 style.9 In-Reply-To: <20010720144839.F65677@sunbay.com> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG 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) { > flag &= ~SLC_LEVELBITS; > flag |= SLC_NOSUPPORT; > slctab[(int)func].current.flag = flag; > } else { > ... > } > > Should that be written as shown? Yes (except "(int)func" looks like it is to break the warning about using a signed char for an index). Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message