From owner-freebsd-doc@FreeBSD.ORG Fri May 23 11:06:30 2003 Return-Path: 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 A080D37B401; Fri, 23 May 2003 11:06:30 -0700 (PDT) Received: from arthur.nitro.dk (port324.ds1-khk.adsl.cybercity.dk [212.242.113.79]) by mx1.FreeBSD.org (Postfix) with ESMTP id 994AB43F85; Fri, 23 May 2003 11:06:29 -0700 (PDT) (envelope-from simon@arthur.nitro.dk) Received: by arthur.nitro.dk (Postfix, from userid 1000) id 260E510BF81; Fri, 23 May 2003 20:06:28 +0200 (CEST) Date: Fri, 23 May 2003 20:06:28 +0200 From: "Simon L. Nielsen" To: Marc Fonvieille Message-ID: <20030523180624.GA65833@nitro.dk> References: <200305231720.h4NHK72A016136@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="OXfL5xGRrasGEqWY" Content-Disposition: inline In-Reply-To: <200305231720.h4NHK72A016136@freefall.freebsd.org> User-Agent: Mutt/1.5.4i cc: freebsd-doc@FreeBSD.org Subject: Re: docs/52547: [patch] Cleanup of laptop article X-BeenThere: freebsd-doc@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Documentation project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 May 2003 18:06:31 -0000 --OXfL5xGRrasGEqWY Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2003.05.23 10:20:07 -0700, Marc Fonvieille wrote: > The following reply was made to PR docs/52547; it has been noted by GNATS. >=20 > On Thu, May 22, 2003 at 01:45:04AM +0200, Simon L.Nielsen wrote: > > General cleanup of the laptop article : > >=20 > > * Add id attributes to all sect1 tags > =20 > This is far to be mandatory on a so little article. Hmm, from following the this mailling list I got the impression that the section id tags was a good idea on all books/articles, since it makes sensible filenames when doing html-split output ? > > * s/FreeBSD/&os;/g > =20 > This tag should be used on new submission, but a full commit does not > worth since it brings nothing new to the reader. Ok. > > * Add application tags around XFree86 references > > * Minor markup cleanups... Hope they right > > * Refer to more files/programs using manual page references > > * Remove some redundant words/sentences (inspired by FDP Writing style= section) > >=20 > > Note: This article is in need of a whitespace cleanup. > > >How-To-Repeat: > > >Fix: > > --- doc-article-laptop-cleanup.patch begins here --- > [...] > > =20 > > - > > - Option "Emulate3Buttons" > > - > > + Option "Emulate3Buttons" > =20 > It's not good: no whitespace/wrapline change with content change. > And this one is quite useless. Doh, yes. I should have remembered that. > =20 > [...] > > - cards is in the file /etc/defaults/pccard.conf. =20 > > + cards is in &man.pccard.conf.5;. > =20 > /etc/defaults/pccard.conf is always up to date, it's not the case of the > manual page. I have often seen manual pages refered to instead of the files they document (at least in other manual pages). This was the main reason for this change, and a few other similar changes in the patch. > =20 > > Look through it, and preferably buy cards listed there. Cards = not > > listed may also work as generic devices: in=20 > > particular most modems (16-bit) should work fine, provided they= =20 > > @@ -128,39 +126,38 @@ > > card, remove irq 5 (otherwise you may experience hangs when you= =20 > > insert a card). Check also the available memory slots; if your= =20 > > card is not being detected, try changing it to one of the other= =20 > > - allowed values (listed in the manual page &man.pccardc.8;). > > + allowed values (listed in &man.pccardc.8;). > =20 > manual page was good there. This change was inspired by the 'Avoid redundant phrases' part of the "Writing style" section of FDP... Is I reading that section wrong, or is the FDP Primer just not updated with current pratice ? Thanks, for taking the time to comment so detailed on the patch! It helps me a lot in getting to know all the do's and don'ts about the documentation. --=20 Simon L. Nielsen --OXfL5xGRrasGEqWY Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2 (FreeBSD) iD8DBQE+zmMg8kocFXgPTRwRAi03AJ983XhTTwacemzXRHGL/IwPabnAIQCeL7VF 2d0eWAO2XEhBVlN7DVYofHk= =90dn -----END PGP SIGNATURE----- --OXfL5xGRrasGEqWY--