Date: Thu, 31 May 2012 10:15:37 -0700 From: Sean Bruno <seanbru@yahoo-inc.com> To: Warren Block <wblock@wonkity.com> Cc: "freebsd-net@freebsd.org" <freebsd-net@freebsd.org> Subject: Re: bce(4) man page updates Message-ID: <1338484537.2985.8.camel@powernoodle-l7.corp.yahoo.com> In-Reply-To: <alpine.BSF.2.00.1205310628200.81499@wonkity.com> References: <1338424909.9051.12.camel@powernoodle-l7.corp.yahoo.com> <alpine.BSF.2.00.1205310628200.81499@wonkity.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 2012-05-31 at 05:46 -0700, Warren Block wrote: > 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)." Indeed. Much more gooder wordingness. Updated! http://people.freebsd.org/~sbruno/bce_man.txt Sean
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1338484537.2985.8.camel>