From owner-cvs-all Sun May 12 13:52:28 2002 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 9124137B403; Sun, 12 May 2002 13:52:25 -0700 (PDT) Received: (from luigi@localhost) by freefall.freebsd.org (8.11.6/8.11.6) id g4CKqPG57056; Sun, 12 May 2002 13:52:25 -0700 (PDT) (envelope-from luigi) Message-Id: <200205122052.g4CKqPG57056@freefall.freebsd.org> From: Luigi Rizzo Date: Sun, 12 May 2002 13:52:25 -0700 (PDT) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sbin/ipfw ipfw.c X-FreeBSD-CVS-Branch: HEAD 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 luigi 2002/05/12 13:52:25 PDT Modified files: sbin/ipfw ipfw.c Log: Main functional change is the implementation of matching of MAC header fields as discussed in the commit to ip_fw.c:1.186 On top of this, a ton of non functional changes to clean up the code, write functions to replace sections of code that were replicated multiple times (e.g. the printing or matching of flags and options), splitting long sections of inlined code into separate functions, and the like. I have tested the code quite a bit, but some typos (using one variable in place of another) might have escaped. The "embedded manpage" is a bit inconsistent, but i am leaving fixing it for later. The current format makes no sense, it is over 40 lines long and practically unreadable. We can either split it into sections ( ipfw -h options , ipfw -h pipe , ipfw -h queue ...) or remove it altogether and refer to the manpage. Revision Changes Path 1.122 +705 -744 src/sbin/ipfw/ipfw.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message