On Sun, 2007-09-09 at 13:46 +0800, Sepherosa Ziehau wrote:

> Updated patch is at:
> http://leaf.dragonflybsd.org/~sephe/fwcutter1.diff
> 
> Please review it, thanks.

I think Pavel suggested you collapse it to:

#ifdef __DragonFly__
#include <sys/endian.h>
#define bswap_16(v)     bswap16((v))
#define bswap_32(v)     bswap32((v))
#else
#include <byteswap.h>
#endif

And maybe all BSDs have sys/endian.h so you could do #ifdef __BSD__ or
something?


I would, however, keep the name thing out of the same ifdef and under
DragonFly to indicate that it is what dfly wants at this time.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to