[PATCH 6/9] Eliminate MAXSCREENS-sized CursorScreenKey array.

Jamey Sharp jamey at minilop.net
Mon Sep 19 22:51:09 PDT 2011


On Tue, Sep 20, 2011 at 03:30:31PM +1000, Peter Hutterer wrote:
> i'd remove the space before '(' but otherwise

Are you talking about this hunk?

> On Sat, Sep 17, 2011 at 03:22:32AM -0500, Jamey Sharp wrote:
> > -extern _X_EXPORT DevPrivateKeyRec cursorScreenDevPriv[MAXSCREENS];
> > -#define CursorScreenKey(pScreen) (cursorScreenDevPriv + (pScreen)->myNum)
> > +extern _X_EXPORT DevScreenPrivateKeyRec cursorScreenDevPriv;
> > +#define CursorScreenKey (&cursorScreenDevPriv)

'cause there has to be whitespace between CursorScreenKey and its
definition. That isn't a parameterized macro.

> Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Thanks!
Jamey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110919/3aeadc7e/attachment.pgp>


More information about the xorg-devel mailing list