On Mon, May 14, 2007 at 10:44:37AM -0400, Greg Troxel wrote: > gnuradio/trunk/usrp/host/lib/legacy/usrp_bytesex.h includes byteswap.h, > which appears to be a Linux-specific header file. I looked briefly, and > didn't see any POSIX/SUSv3 definitions of these macros. I just fixed > this to build on NetBSD, which I suspect also fixes it on several other > non-Linux systems. > > > Also, the code uses 'short int' and 'int', which is nonportable. It > should be using the POSIX fixed-width types.
Agreed. > I wonder if the use of swap is really right - this seems to be about > host to USRP, but perhaps the USRP is always little endian? It > definitely needs more comments if that's what's going on. The USRP is always little endian. We didn't want to handle both cases in the verilog, and at least at this point in history, most of our hosts are little endian. Eric _______________________________________________ Discuss-gnuradio mailing list Discuss-gnuradio@gnu.org http://lists.gnu.org/mailman/listinfo/discuss-gnuradio
