Date: Sat, 21 Jul 2001 01:38:06 +1000 (EST) From: Bruce Evans <bde@zeta.org.au> To: 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: <Pine.BSF.4.21.0107210133070.10938-100000@besplex.bde.org> In-Reply-To: <20010720144839.F65677@sunbay.com>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0107210133070.10938-100000>