From owner-cvs-all Mon Oct 29 3:29:59 2001 Delivered-To: cvs-all@freebsd.org Received: from tao.org.uk (genius.tao.org.uk [212.135.162.51]) by hub.freebsd.org (Postfix) with ESMTP id 7766737B403; Mon, 29 Oct 2001 03:29:50 -0800 (PST) Received: by tao.org.uk (Postfix, from userid 100) id 2198CCC; Mon, 29 Oct 2001 11:29:29 +0000 (GMT) Date: Mon, 29 Oct 2001 11:29:28 +0000 From: Josef Karthauser To: "Andrew R. Reiter" Cc: Josef Karthauser , cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: ipfw.c -- (was: cvs commit: src/sys/netinet ip_fw.h) Message-ID: <20011029112928.C867@tao.org.uk> References: <200110290441.f9T4fSY68423@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-md5; protocol="application/pgp-signature"; boundary="8NvZYKFJsRX2Djef" Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: ; from arr@watson.org on Mon, Oct 29, 2001 at 12:11:24AM -0500 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 --8NvZYKFJsRX2Djef Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 29, 2001 at 12:11:24AM -0500, Andrew R. Reiter wrote: >=20 > Sorry to reply to ipfw.c commits this way, but while there have been style > fixes, I don't think all have been fixed. For example, in show_ipfw(), > there is char timestr[30]; which is found in the middle of an if block. > IIRC, isn't this a style problem as well? Also, I would imagine that it's > just poor design to have to declare variables in the middle of a block, > whether it be an if-block, for-block, etc.. >=20 Maybe :) Feel free to commit fixes to -current if you care :). The style changes I've made to -current are ones that were already in the -stable version, but would have been backed out during MFC. There are many many more improvements that could be made, but my goal is to sync the code bases as much as possible so that the "normal" development process can continue - I'm not an ipfw developer myself. Joe --8NvZYKFJsRX2Djef Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (FreeBSD) Comment: For info see http://www.gnupg.org iEYEARECAAYFAjvdPZgACgkQXVIcjOaxUBa8aQCgmCDm7XHMq0Vbjn1vsU2XqmU4 VtgAoKbApr2mYx4Hc/ln/E4w7lGWmHTA =BDzD -----END PGP SIGNATURE----- --8NvZYKFJsRX2Djef-- To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message