Re: CVS commit: src/share/man/man9

[Paul Goyette <paul%whooppee.com@localhost>]

On Tue, 4 May 2010, Izumi Tsutsui wrote:

> > Modified Files:
> >         src/share/man/man9: byteorder.9
> >
> > Log Message:
> > Reference bswap(3). Improvements in the HISTORY section.
> :
> > cvs rdiff -u -r1.4 -r1.5 src/share/man/man9/byteorder.9
>
> > > -These functions were introduced to handle PCI bus master devices which assumed
> > > -host's memory used to pass integer parameters via DMA transfer was
> > > -always little endian even on big endian systems.
> :
> > > +The functions were originally introduced to handle
> > > +.Tn PCI
> > > +bus master devices, which assumed little endian byte order in
> > > +.Tn DMA
> > > +transfers, even on big endian systems.
>
> There are a few PCI bus master devices that are big endian aware
> (epic(4) for example), so I don't think it's appropriate to add
> commas around the which clause.

I agree that the first comma should go, but the second one (before 
"even") should remain.


-------------------------------------------------------------------------
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:       |
| Customer Service | FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com    |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
| Kernel Developer |                          | pgoyette at netbsd.org  |
-------------------------------------------------------------------------