From owner-freebsd-doc@FreeBSD.ORG  Tue Sep 21 04:19:57 2004
Return-Path: <owner-freebsd-doc@FreeBSD.ORG>
Delivered-To: freebsd-doc@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 837EE16A4CE
	for <freebsd-doc@freebsd.org>; Tue, 21 Sep 2004 04:19:57 +0000 (GMT)
Received: from igloo.linux.gr (igloo.linux.gr [62.1.205.36])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 8D81243D4C
	for <freebsd-doc@freebsd.org>; Tue, 21 Sep 2004 04:19:55 +0000 (GMT)
	(envelope-from keramida@ceid.upatras.gr)
Received: from igloo.linux.gr (localhost [127.0.0.1])i8L4JKxQ000544;
	Tue, 21 Sep 2004 07:19:21 +0300
Received: (from keramida@localhost)
	by igloo.linux.gr (8.12.11/8.12.11/Debian-5) id i8L4JK7Q000539;
	Tue, 21 Sep 2004 07:19:20 +0300
X-Authentication-Warning: igloo.linux.gr: keramida set sender to
	keramida@ceid.upatras.gr using -f
Date: Tue, 21 Sep 2004 07:19:19 +0300
From: Giorgos Keramidas <keramida@ceid.upatras.gr>
To: Brad Davis <so14k@so14k.com>
Message-ID: <20040921041919.GA32654@igloo.linux.gr>
References: <20040918.161309.35654157.hrs@eos.ocn.ne.jp>
	<20040920110628.GA2493@submonkey.net>
	<20040920133025.GB38865@orion.daedalusnetworks.priv>
	<200409201659.00842.so14k@so14k.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <200409201659.00842.so14k@so14k.com>
X-Hellug-MailScanner: Found to be clean
X-Hellug-MailScanner-SpamCheck: not spam, SpamAssassin (score=-4.199,
	required 5, autolearn=not spam, AWL 0.70, BAYES_00 -4.90)
X-MailScanner-From: keramida@ceid.upatras.gr
cc: freebsd-doc@freebsd.org
Subject: Re: New firewall section (was: Re: HEADS UP: doc/ slush begins)
X-BeenThere: freebsd-doc@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: Documentation project <freebsd-doc.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-doc>,
	<mailto:freebsd-doc-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-doc>
List-Post: <mailto:freebsd-doc@freebsd.org>
List-Help: <mailto:freebsd-doc-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-doc>,
	<mailto:freebsd-doc-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 21 Sep 2004 04:19:57 -0000

On 2004-09-20 16:59, Brad Davis <so14k@so14k.com> wrote:
> > : +      <para>The IPFW sample rule set (found in
> > : +        <filename>/etc/rc.firewall</filename>) delivered in the basic [...]
> >
> > What are the ``service ports'' referred to here?
>
> This does need better wording.. I can't come up with anything right now.

I'll see if I can come up with something.

> > : +      <para>The OpenBSD PF user's guide is here:
> > : +        <ulink url="http://www.openbsd.org/faq/pf/index.html"></ulink>.
> > : +        </para>
> >
> > Please trim the unnecessary whitespace here too.
>
> Well the period is at line 71, so I pushed the </para> down so I
> wouldn't go over the 72 line rule.

IMHO, it's ok to keep longer lines in the source when there is no
obvious way to split the text.

> Also I have encountered an undefined &man.ipnat.8

That should have been &man.ipnat.1;.  I was writing the comments in a
haste and didn't check my entities, but we can easily fix that :-)