Date: Thu, 31 May 2012 06:46:20 -0600 (MDT) From: Warren Block <wblock@wonkity.com> To: sbruno@freebsd.org Cc: "freebsd-net@freebsd.org" <freebsd-net@freebsd.org> Subject: Re: bce(4) man page updates Message-ID: <alpine.BSF.2.00.1205310628200.81499@wonkity.com> In-Reply-To: <1338424909.9051.12.camel@powernoodle-l7.corp.yahoo.com> References: <1338424909.9051.12.camel@powernoodle-l7.corp.yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 30 May 2012, Sean Bruno wrote: > Was trolling around inside of bce(4) and the Broadcom docs today and > made the following update to the man page. Thoughts? > > Sean > > http://people.freebsd.org/~sbruno/bce_man.txt Nice! Just minor suggestions. The two "whether or not" sentences ought to be just "Enable/Disable ..." like the rest. A couple of "Cannot be assigned the value 0" sentences can be simpler as "Cannot be set to 0" if that doesn't change the meaning. "Valid values are in the range from 0-256." can be simpler as "Values from 0-256 are valid." Many "The default value is" which could be simplified to "The default is", or added to the end of a previous sentence as an aside: "Values from 0-100 are valid (default 18)."
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.1205310628200.81499>