From owner-freebsd-audit Tue Jun 5 10:39: 7 2001 Delivered-To: freebsd-audit@freebsd.org Received: from ringworld.nanolink.com (ringworld.nanolink.com [195.24.48.13]) by hub.freebsd.org (Postfix) with SMTP id 1FDA037B408 for ; Tue, 5 Jun 2001 10:39:04 -0700 (PDT) (envelope-from roam@orbitel.bg) Received: (qmail 9911 invoked by uid 1000); 5 Jun 2001 17:37:53 -0000 Date: Tue, 5 Jun 2001 20:37:53 +0300 From: Peter Pentchev To: Dima Dorfman Cc: audit@FreeBSD.org Subject: Re: rarpd(8) cleanup patch Message-ID: <20010605203753.D587@ringworld.oblivion.bg> Mail-Followup-To: Dima Dorfman , audit@FreeBSD.org References: <20010604161628.A20595@ringworld.oblivion.bg> <20010605012652.B06E63E31@bazooka.unixfreak.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20010605012652.B06E63E31@bazooka.unixfreak.org>; from dima@unixfreak.org on Mon, Jun 04, 2001 at 06:26:52PM -0700 Sender: owner-freebsd-audit@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG OK, points taken. The new patches are at: http://people.FreeBSD.org/~roam/bsd/rarpd/usr.sbin-rarpd.patch http://people.FreeBSD.org/~roam/bsd/rarpd/usr.sbin-rarpd-d.patch G'luck, Peter -- No language can express every thought unambiguously, least of all this one. On Mon, Jun 04, 2001 at 06:26:52PM -0700, Dima Dorfman wrote: > Peter Pentchev writes: > > At http://people.FreeBSD.org/~roam/bsd/rarpd/usr.sbin-rarpd.patch > > there is a patch that does the following: > > - remove 'register' keywords; > > - add 'const' to a couple of strings; > > - add prototypes for several missed functions; > > - several int/u_int/unsigned/long/format cleanups for WARNS=2; > > - mdoc(7) cleanup for the rarpd.8 manpage. > > Couple of comments: > > * Kris said WARNS should be set like this: "WARNS?= 2" to allow overriding. > * main() doesn't have to be prototyped. GCC complains about it right > now; I've posted a patch to it which will shut it up in this case, and > obrien said he'd commit it but apparently hasn't gotten around to it > yet. > * The function prototypes at the top should be in alphabetical order. > * style(9) says that there shouldn't be a space after a cast. I.e., this: > > (int) sizeof(fish) > > is wrong. It should be: > > (int)sizeof(fish) > > Other than that, this looks good. > > > Functions and prototypes were left in K&R style, although an ANSIfication > > would make the next change a lot easier. Should I ansify the lot? > > I'd say so. ANSI is preferred, so it isn't worth the trouble to > support K&R if you have to go out of your way. stlye(9) says you > shouldn't break support for it gratuitously, but I think support for > varargs.h in new code is a little too much :-). > > > In addition, http://people.FreeBSD.org/~roam/bsd/rarpd/usr.sbin-rarpd-d.patch > > adds a command-line flag, -d, that makes rarpd(8) send diagnostic messages > > to stdout, error messages to stderr. This is quite useful when running > > rarpd(8) under some kind of service monitoring script, e.g. supervise/svscan > > from DJB's daemontools package. > > This one looks good, too, the above comments notwithstanding. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message