Keith Packard wrote:
> On Mon, 11 Oct 2010 11:16:54 -0400, Adam Jackson <[email protected]> wrote:
>
>> +#define CallCallbacks(x, y) \
>> + if ((*(x))) CallCallbacks(x, y);
>> +
>
> I know the patch is cute like this, but it seems like simply making
> CallCallbacks an inline function would be cleaner.
CallCallbacks is just one more check than the above, so the only downside
I see is that _CallCallbacks then has to go from a static function to
becoming an exported part of the ABI.
--
-Alan Coopersmith- [email protected]
Oracle Solaris Platform Engineering: X Window System
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
