> From: Keith Packard <kei...@keithp.com> > Date: Tue, 9 Dec 2014 09:30:57 -0800 > > When the local types used to walk the DBE request were changed, this > changed the type of the parameter passed to the DDX SwapBuffers API, > but there wasn't a matching change in the API definition. > > At this point, with the API frozen, I just stuck a new variable in > with the correct type. Because we've already bounds-checked nStuff to > be smaller than UINT32_MAX / sizeof(DbeSwapInfoRec), we know it will > fit in a signed int without overflow.
Hmm, unless the comment: /* We don't have DDX support for DBE anymore */ dbe/dbe.c:DbeExtensionInit() is a lie, I believe this isn't really part of the external API. > Signed-off-by: Keith Packard <kei...@keithp.com> > --- > dbe/dbe.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) _______________________________________________ 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