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

Eric Anholt eric at anholt.net
Mon Apr 24 20:16:10 UTC 2017


Keith Packard <keithp at keithp.com> writes:

> [ Unknown signature status ]
> 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?

It doesn't find anything.
-------------- 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/20170424/aa96d1ae/attachment.sig>


More information about the xorg-devel mailing list