Date: Fri, 23 May 2003 20:06:28 +0200 From: "Simon L. Nielsen" <simon@nitro.dk> To: Marc Fonvieille <blackend@FreeBSD.org> Cc: freebsd-doc@FreeBSD.org Subject: Re: docs/52547: [patch] Cleanup of laptop article Message-ID: <20030523180624.GA65833@nitro.dk> In-Reply-To: <200305231720.h4NHK72A016136@freefall.freebsd.org> References: <200305231720.h4NHK72A016136@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--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 > > - <programlisting> > > - Option "Emulate3Buttons" > > - </programlisting> > > + <programlisting>Option "Emulate3Buttons"</programlisting> > =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 <filename>/etc/defaults/pccard.conf</filen= ame>. =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 <quote>generic</quote> 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--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030523180624.GA65833>