[PATCH] Remove incorrect & in swap_uint32

Jamey Sharp jamey at minilop.net
Wed Sep 28 14:31:12 PDT 2011


Ow, harsh.

Reviewed-by: Jamey Sharp <jamey at minilop.net>

On Wed, Sep 28, 2011 at 05:27:45PM -0400, Matt Turner wrote:
> Caused by commit 893e86a4, and hidden by the (char *) cast.
> 
> Signed-off-by: Matt Turner <mattst88 at gmail.com>
> ---
>  include/misc.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/include/misc.h b/include/misc.h
> index 1fea73e..180b3c1 100644
> --- a/include/misc.h
> +++ b/include/misc.h
> @@ -277,7 +277,7 @@ static inline void __builtin_constant_p(int x)
>  /* byte swap a 32-bit value */
>  static inline void swap_uint32(uint32_t *x)
>  {
> -	char n = ((char *) &x)[0];
> +	char n = ((char *) x)[0];
>  	((char *) x)[0] = ((char *) x)[3];
>  	((char *) x)[3] = n;
>  	n = ((char *) x)[1];
> -- 
> 1.7.3.4
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110928/7e3a71f4/attachment.pgp>


More information about the xorg-devel mailing list