On Sun, Mar 27, 2011 at 21:32:10 +0200, Matthieu Herrb wrote:

> On Sun, Mar 27, 2011 at 09:06:08PM +0200, Julien Cristau wrote:
> > This seems dangerous if you're, say, building an X server that only
> > supports XI 2.0 against the XI 2.1 headers.  Should probably be
> > "if (stuff->data >= ARRAY_SIZE(SProcIVector)) ..."
> > (looks like there's no ARRAY_SIZE macro outside of glx, but you get the
> > idea)
> > 
> 
> Well it matches the test done for the non-swapping case just 10 lines
> above:
> 
I know.  It just means they're both wrong, IMO.

Cheers,
Julien
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to