[PATCH xserver 1/3] Rewrite the byte swapping macros.

Keith Packard keithp at keithp.com
Sat Apr 22 08:57:06 UTC 2017


Eric Anholt <eric at anholt.net> writes:

> The reason to think so is that the previous code had this weird stuff to
> explicitly handle unaligned.  I rip out the unaligned support in patch
> 3, because I think it's not needed.

Maybe consider adding -Wcast-align to CFLAGS, get rid of the memcpy and
see if the compiler catches the code doing anything wrong?

-- 
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20170422/753c2374/attachment.sig>


More information about the xorg-devel mailing list