[PATCH 1/5] Use internal temp variable for swap macros

Matt Turner mattst88 at gmail.com
Thu Aug 4 15:11:58 PDT 2011


This patch was way too big to make it to the mailing list, so you can
see it here

http://cgit.freedesktop.org/~mattst88/xserver/commit/?h=bswap-cleanup&id=039d90b2a6a68496ef63f4b7e557c41b3bc82379

I used Coccinelle and a simple semantic patch to remove the second
argument from swap[ls]:
---
@@
expression x, n;
@@

- swapl(x, n)
+ swapl(x)

@@
expression x, n;
@@

- swaps(x, n)
+ swaps(x)
---

Left over 'register int n'-type variables were removed by hand, so
that no unused variable warnings were added.

Matt


More information about the xorg-devel mailing list