[PATCH] xace: micro-optimize callbacks

Adam Jackson ajax at redhat.com
Mon Oct 11 13:15:45 PDT 2010


On Mon, 2010-10-11 at 11:26 -0700, Keith Packard wrote:
> On Mon, 11 Oct 2010 11:16:54 -0400, Adam Jackson <ajax at redhat.com> 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.

That would have the pleasant side effect of fixing all callers.  Revised
patch shortly.

- ajax

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101011/e6c89bef/attachment.pgp>


More information about the xorg-devel mailing list