From owner-freebsd-audit Wed Jun 27 12:40:21 2001 Delivered-To: freebsd-audit@freebsd.org Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by hub.freebsd.org (Postfix) with ESMTP id EF13737B406; Wed, 27 Jun 2001 12:40:09 -0700 (PDT) (envelope-from brdavis@odin.ac.hmc.edu) Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.11.0/8.11.0) id f5RJe9C29463; Wed, 27 Jun 2001 12:40:09 -0700 Date: Wed, 27 Jun 2001 12:40:09 -0700 From: Brooks Davis To: Ruslan Ermilov Cc: net@FreeBSD.ORG, audit@FreeBSD.ORG Subject: Re: review request: network interface cloning Message-ID: <20010627124009.D12739@Odin.AC.HMC.Edu> References: <20010626144313.A7909@Odin.AC.HMC.Edu> <20010627111159.E2097@sunbay.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-md5; protocol="application/pgp-signature"; boundary="osDK9TLjxFScVI/L" Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20010627111159.E2097@sunbay.com>; from ru@FreeBSD.ORG on Wed, Jun 27, 2001 at 11:12:00AM +0300 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 --osDK9TLjxFScVI/L Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Thanks for the review. On Wed, Jun 27, 2001 at 11:12:00AM +0300, Ruslan Ermilov wrote: > Please don't use hard sentence breaks. I'm, not sure what you mean by this. Do you mean, don't use ". " or what? I've moved to split sentences by lines. > Especially with the long getopt() list, it might be a good idea to put > `case' statements in the same order as they were in getopt() call. I've reordered the switch statement to be ordered like getopt. It's not entierly right and Sheldon is correct that usage and the SYNOPSIS are also misorderd, but I'm not going to fix that just now. There are lots of problems with ifconfig and fixing them isn't really the point of this patch. I'm actually thinking of taking a look at adding our local extensions to the NetBSD ifconfig source and moving to that since they have actually maintained it. > You need to decide, memset(3) or bzero(3). :-) memset looks more common so I'm using it now. Some bzeros remain in unrelated parts of the code. > > + err =3D (*ifc->ifc_create)(ifc, &unit); > > + if (err !=3D 0) > > + return (err); > > + > if (err) > return (err); >=20 > is more traditional :-) But err isn't a boolean and the example in style(9) is: error =3D function(a1, a2); if (error !=3D 0) exit(error); The bug I documented in the gif manpage was caught and squashed so that has been removed. I've updated the diff at: http://people.freebsd.org/~brooks/patches/gif.diff -- Brooks --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --osDK9TLjxFScVI/L Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.4 (GNU/Linux) Comment: For info see http://www.gnupg.org iD8DBQE7OjaYXY6L6fI4GtQRAv9SAJ9yM2ydWxKe3mWdUrNSPat/0jKTGgCaA22p z1cbc1x7Dj2iOuUZBwQqriY= =Tw8F -----END PGP SIGNATURE----- --osDK9TLjxFScVI/L-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message