From owner-cvs-all Thu Oct 3 6: 0:15 2002 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6527137B433; Thu, 3 Oct 2002 06:00:11 -0700 (PDT) Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by mx1.FreeBSD.org (Postfix) with ESMTP id 826C543E91; Thu, 3 Oct 2002 06:00:09 -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 WAA22937; Thu, 3 Oct 2002 22:59:56 +1000 Date: Thu, 3 Oct 2002 23:09:35 +1000 (EST) From: Bruce Evans X-X-Sender: bde@gamplex.bde.org To: Robert Drehmel Cc: Robert Drehmel , , Subject: Re: cvs commit: src/lib/libc/net inet.3 In-Reply-To: <20020816115127.A3354@alpha.develop.ferrari.net> Message-ID: <20021003225719.X3806-100000@gamplex.bde.org> 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 Long ago, On Fri, 16 Aug 2002, Robert Drehmel wrote: > On Fri, Aug 16, 2002 at 06:07:57PM +1000, Bruce Evans wrote: > > On Thu, 15 Aug 2002, Robert Drehmel wrote: > > > > > robert 2002/08/15 13:33:44 PDT > > > > > > Modified files: > > > lib/libc/net inet.3 > > > Log: > > > Put each function argument on its own line to keep lines shorter > > > than 80 columns. > > > > > > Revision Changes Path > > > 1.24 +6 -1 src/lib/libc/net/inet.3 > > > > The limit of 9 macro args was fixed a year or two ago, and backslash/newline > > in man pages has worked for much longer, so man pages should be changed > > away from using the ugly .Fo ... .Fc construct and not towards it. I noticed a few more conversions to .Fo ... .Fc in sigaction.2 and sigprocmask.2. Some of the corresponding C protypes in became multi-line (because addition of __restrict expanded them). > People seem to have differing opinions about that. ru basically > told me that using ".Fo .Fa* .Fc" or ".Fn" is ok. I am now a bit > confused about what construct to use, i.e. what style is preferred. I think it is OK for old code but shouldn't be used for new code. > > Especially when you change the corresponding non-ugly C construct in the > > opposite direction :-). (Some people prefer old-style ANSI function > > declarations (with one physical line per parameter) to new-style ANSI > > function declarations (with all parameters on 1 longical line). > > In my opinion, that looks especially concise for functions with many > arguments like vm_map_lookup() in sys/vm/vm_map.c. I think you mean "especaully verbose". vm_map_lookup() is a good bad example. It has a new-style ANSI function declaration but is formatted like an old-style ANSI function function declaration (except for indentation bugs) to provide a place to attach comments. > > The separate lines at least provide a good place to put comments on the > > parameters in C declarations. This advantage doesn't apply to prototypes > > in either C code or man pages.) Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message