From owner-freebsd-doc Wed Aug 18 5:31:12 1999 Delivered-To: freebsd-doc@freebsd.org Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21]) by hub.freebsd.org (Postfix) with ESMTP id B447F14DC2 for ; Wed, 18 Aug 1999 05:31:10 -0700 (PDT) (envelope-from gnats@FreeBSD.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.9.3/8.9.2) id FAA63894; Wed, 18 Aug 1999 05:30:01 -0700 (PDT) (envelope-from gnats@FreeBSD.org) Date: Wed, 18 Aug 1999 05:30:01 -0700 (PDT) Message-Id: <199908181230.FAA63894@freefall.freebsd.org> To: freebsd-doc@freebsd.org Cc: From: Tim Vanderhoek Subject: Re: docs/13218: [HUGE PATCH] corrections for manpages Reply-To: Tim Vanderhoek Sender: owner-freebsd-doc@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org The following reply was made to PR docs/13218; it has been noted by GNATS. From: Tim Vanderhoek To: "Alexey M. Zelkin" Cc: FreeBSD-gnats-submit@freebsd.org Subject: Re: docs/13218: [HUGE PATCH] corrections for manpages Date: Wed, 18 Aug 1999 08:30:27 -0400 On Wed, Aug 18, 1999 at 10:44:13AM +0400, Alexey M. Zelkin wrote: > > I see that Chris Costelo already works on this task. There lot's > style and look corrections for FreeBSD manpages. I don't know why, but patch > is again huge :-)) Probably manpages last time were reviewed long time ago ... I'm not sure what the purpose of the following hunk is... Index: share/man/man4/man4.i386/meteor.4 =================================================================== RCS file: /usr/local/CVSROOT/src/share/man/man4/man4.i386/meteor.4,v retrieving revision 1.8 diff -c -r1.8 meteor.4 [...] *** 71,77 **** behavior is to not deallocate pages. .Pp .Em options "METEOR_DEALLOC_ABOVE=xxx" ! deallocate all pages above the specified number. The default action is to not deallocate above any pages. .Pp 3) Make and install the kernel. --- 69,75 ---- behavior is to not deallocate pages. .Pp .Em options "METEOR_DEALLOC_ABOVE=xxx" ! deallocate all pages above the specified number. The default action is to not deallocate above any pages. .Pp 3) Make and install the kernel. *************** If the only change is to single-space after the period, I strongly object. It looks like you are trying to make the line fit some minimum width. If this is the case, I'd rather you simply started a newline after the period. The advantage to this is that it makes future diffs smaller and more readable. Having simpler diffs, more specific, diffs is highly desirable since it makes the job of the translators easier. Also, starting each sentence on a newline allows the manpages to have the correct number of spaces used after each period (except in certain broken instances). Speaking of translators...if this was done by script, you may want to supply a copy of the script with this PR. It might (or might not) be useful. Also, we don't have a program named bsd2dos. Some of the .Xr's you've added don't exist. This doesn't bother me too much, but.... -- This is my .signature which gets appended to the end of my messages. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-doc" in the body of the message