>> --- a/glx/createcontext.c >> +++ b/glx/createcontext.c >> @@ -320,7 +320,7 @@ __glXDisp_CreateContextAttribsARB(__GLXclientState * cl, >> GLbyte * pc) >> ctx->id = req->context; >> ctx->share_id = req->shareList; >> ctx->idExists = True; >> - ctx->isCurrent = False; >> + ctx->currentClient = False; >> ctx->isDirect = req->isDirect; >> ctx->hasUnflushedCommands = False; >> ctx->renderMode = GL_RENDER;
Admittedly, I don't know glx internals, so I could be totally off base here. But the only declaration of currentClient I find is a pointer, not a boolean, which would make False conceptually wrong, even though if (as seems likely) False is #defined to 0 it's actually valid C. Should this maybe be 0 instead of False? /~\ The ASCII Mouse \ / Ribbon Campaign X Against HTML mo...@rodents-montreal.org / \ Email! 7D C8 61 52 5D E7 2D 39 4E F1 31 3E E8 B3 27 4B _______________________________________________ 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