On Fri, 14 May 2010 17:45:00 -0700, Jamey Sharp <[email protected]> wrote:
> The comment suggests that after unwrapping, pGC->funcs->ChangeClip is
> miChangeClip. I just tried tracing through the layers of function
> pointers and I *think* that's true. After miChangeClip returns,
> clientClipType can only be either CT_REGION or CT_NONE. So this "fix"
> should be correct. I'd rather see a "default: FatalError(...);" case
> instead of commented-out code, though. Preserving or improving the
> comment would be an added bonus.
The warning was due to the field width of pGC->clientClipType, which is
only two bits wide, and so it's fairly pointless to check for anything
other than CT_NONE, CT_PIXMAP or CT_REGION. And, as you say, the mi code
will have converted CT_PIXMAP into CT_REGION by the time the dmx code
sees it. I figured not changing the behavior in code I wasn't going to
test was a better plan than sticking in an 'abort' though.
> I'd think the config/udev change should be replaced by Julien's patch,
> which has been reviewed now.
Yup.
> For this remaining one-line change ;-) from your git tree:
>
> hw/dmx/glxProxy/glxcmds.c:
> - if (fbconfigId == NULL && visual == NULL) {
> + if (fbconfigId == None && visual == None) {
>
> Reviewed-by: Jamey Sharp <[email protected]>
Yay! Thanks!
--
[email protected]
pgpYVyp7jts3F.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
