From owner-svn-src-stable-9@FreeBSD.ORG Fri Mar 23 10:55:19 2012 Return-Path: Delivered-To: svn-src-stable-9@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id D8BC11065673; Fri, 23 Mar 2012 10:55:19 +0000 (UTC) (envelope-from dim@FreeBSD.org) Received: from svn.freebsd.org (svn.freebsd.org [IPv6:2001:4f8:fff6::2c]) by mx1.freebsd.org (Postfix) with ESMTP id B271B8FC12; Fri, 23 Mar 2012 10:55:19 +0000 (UTC) Received: from svn.freebsd.org (localhost [127.0.0.1]) by svn.freebsd.org (8.14.4/8.14.4) with ESMTP id q2NAtJvm070984; Fri, 23 Mar 2012 10:55:19 GMT (envelope-from dim@svn.freebsd.org) Received: (from dim@localhost) by svn.freebsd.org (8.14.4/8.14.4/Submit) id q2NAtJ9p070979; Fri, 23 Mar 2012 10:55:19 GMT (envelope-from dim@svn.freebsd.org) Message-Id: <201203231055.q2NAtJ9p070979@svn.freebsd.org> From: Dimitry Andric Date: Fri, 23 Mar 2012 10:55:19 +0000 (UTC) To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org X-SVN-Group: stable-9 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cc: Subject: svn commit: r233349 - in stable/9/sys: amd64/include i386/include powerpc/include sparc64/include X-BeenThere: svn-src-stable-9@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for only the 9-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Mar 2012 10:55:20 -0000 Author: dim Date: Fri Mar 23 10:55:19 2012 New Revision: 233349 URL: http://svn.freebsd.org/changeset/base/233349 Log: MFC r232745: Add casts to __uint16_t to the __bswap16() macros on all arches which didn't already have them. This is because the ternary expression will return int, due to the Usual Arithmetic Conversions. Such casts are not needed for the 32 and 64 bit variants. While here, add additional parentheses around the x86 variant, to protect against unintended consequences. Modified: stable/9/sys/amd64/include/endian.h stable/9/sys/i386/include/endian.h stable/9/sys/powerpc/include/endian.h stable/9/sys/sparc64/include/endian.h Directory Properties: stable/9/sys/ (props changed) Modified: stable/9/sys/amd64/include/endian.h ============================================================================== --- stable/9/sys/amd64/include/endian.h Fri Mar 23 10:39:50 2012 (r233348) +++ stable/9/sys/amd64/include/endian.h Fri Mar 23 10:55:19 2012 (r233349) @@ -119,8 +119,8 @@ __bswap16_var(__uint16_t _x) __bswap32_const((__uint32_t)(_x)) : __bswap32_var(_x)) #define __bswap16(_x) \ - (__builtin_constant_p(_x) ? \ - __bswap16_const((__uint16_t)(_x)) : __bswap16_var(_x)) + ((__uint16_t)(__builtin_constant_p(_x) ? \ + __bswap16_const((__uint16_t)(_x)) : __bswap16_var(_x))) #define __htonl(x) __bswap32(x) #define __htons(x) __bswap16(x) Modified: stable/9/sys/i386/include/endian.h ============================================================================== --- stable/9/sys/i386/include/endian.h Fri Mar 23 10:39:50 2012 (r233348) +++ stable/9/sys/i386/include/endian.h Fri Mar 23 10:55:19 2012 (r233349) @@ -119,8 +119,8 @@ __bswap16_var(__uint16_t _x) __bswap32_const((__uint32_t)(_x)) : __bswap32_var(_x)) #define __bswap16(_x) \ - (__builtin_constant_p(_x) ? \ - __bswap16_const((__uint16_t)(_x)) : __bswap16_var(_x)) + ((__uint16_t)(__builtin_constant_p(_x) ? \ + __bswap16_const((__uint16_t)(_x)) : __bswap16_var(_x))) #define __htonl(x) __bswap32(x) #define __htons(x) __bswap16(x) Modified: stable/9/sys/powerpc/include/endian.h ============================================================================== --- stable/9/sys/powerpc/include/endian.h Fri Mar 23 10:39:50 2012 (r233348) +++ stable/9/sys/powerpc/include/endian.h Fri Mar 23 10:55:19 2012 (r233349) @@ -124,8 +124,8 @@ __bswap64_var(__uint64_t _x) ((_x << 40) & ((__uint64_t)0xff << 48)) | ((_x << 56))); } -#define __bswap16(x) (__is_constant(x) ? __bswap16_const(x) : \ - __bswap16_var(x)) +#define __bswap16(x) ((__uint16_t)(__is_constant(x) ? __bswap16_const(x) : \ + __bswap16_var(x))) #define __bswap32(x) (__is_constant(x) ? __bswap32_const(x) : \ __bswap32_var(x)) #define __bswap64(x) (__is_constant(x) ? __bswap64_const(x) : \ Modified: stable/9/sys/sparc64/include/endian.h ============================================================================== --- stable/9/sys/sparc64/include/endian.h Fri Mar 23 10:39:50 2012 (r233348) +++ stable/9/sys/sparc64/include/endian.h Fri Mar 23 10:55:19 2012 (r233349) @@ -109,8 +109,8 @@ __bswap64_var(__uint64_t _x) ((_x << 40) & ((__uint64_t)0xff << 48)) | ((_x << 56))); } -#define __bswap16(x) (__is_constant(x) ? __bswap16_const(x) : \ - __bswap16_var(x)) +#define __bswap16(x) ((__uint16_t)(__is_constant(x) ? __bswap16_const(x) : \ + __bswap16_var(x))) #define __bswap32(x) (__is_constant(x) ? __bswap32_const(x) : \ __bswap32_var(x)) #define __bswap64(x) (__is_constant(x) ? __bswap64_const(x) : \