>> --- 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

Reply via email to