On Tue, Aug 10, 2010 at 12:55:38PM +0530, Jayachandran C. wrote: > > I think there is a problem in sys/mips/include/_endian.h > -- > #define __bswap16(x) (__uint16_t)(__is_constant(x) ? \ > __bswap16_const((__uint16_t)x) : __bswap16_var((__uint16_t)x)) > #define __bswap32(x) (__uint32_t)(__is_constant(x) ? \ > __bswap32_const((__uint32_t)x) : __bswap32_var((__uint32_t)x)) > #define __bswap64(x) (__uint64_t)(__is_constant(x) ? \ > __bswap64_const((__uint64_t)x) : __bswap64_var((__uint64_t)x)) > -- > > I'm not sure why the cast is needed, but we should have a braces > around x, unless I'm completely mistaken.
I agree. And around the entire expression too. Stefan _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
