Update to my review request (New content and changes)

[Simon L. Nielsen]

On 2004.06.16 17:53:15 -0400, Tom Rhodes wrote:

I only looked at the manual pages for now...

> New manual pages:

> cm.4

> .Bl -item -compact
> .It
> SMC90c26
> .It
> SMC90c56
> .It
> and SMC90c66 in '56 compatibility mode.

s/and// - IMO since it's a list, not normal text.

...

>.Sh IMPLEMENTATION NOTES
> The
> .Dv IFF_LINK0
> flag is used to switch between
> .Tn RFC 1051
> mode and
> .Tn RFC
> 1201 mode connectivity.
> The default is
> .Tn RFC
> 1201, or category five cable with

RFC is normally not marked up with Tn.  There are also several other
places where .Tn is used, where I at least wouldn't have done that.

> esp.4

> and certain types of Parallel port hardware.

Should that be capital P in Parallel ?

> .Sh AUTHORS
> This manual page was written by
> .An Tom Rhodes Aq trhodes at FreeBSD.org
> in place of porting the original from
> .Nx .

The "in place of" sounds a bit odd to me...

In rc.4 there are a few long (>80 chars) lines.

snc.4 also has a lot of .Tn uses...

-- 
Simon L. Nielsen
FreeBSD Documentation Team
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-doc/attachments/20040617/f6636246/attachment.sig>