GLX_ARB_create_context, which we aspire to support, allows making GL 3.0 or newer contexts current with null current drawables. Strictly this might not be legal for pre-3.0 contexts, but there's no harm in allowing it anyway.
Signed-off-by: Adam Jackson <a...@redhat.com> --- glx/glxcmds.c | 39 +++++++++++++++++---------------------- 1 file changed, 17 insertions(+), 22 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index 67f9509..af36cf1 100644 --- a/glx/glxcmds.c +++ b/glx/glxcmds.c @@ -560,19 +560,14 @@ DoMakeCurrent(__GLXclientState * cl, __GLXdrawable *drawPriv = NULL; __GLXdrawable *readPriv = NULL; int error; - GLuint mask; - /* - ** If one is None and the other isn't, it's a bad match. - */ - - mask = (drawId == None) ? (1 << 0) : 0; - mask |= (readId == None) ? (1 << 1) : 0; - mask |= (contextId == None) ? (1 << 2) : 0; + /* Drawables but no context makes no sense */ + if (!contextId && (drawId || readId)) + return BadMatch; - if ((mask != 0x00) && (mask != 0x07)) { + /* If either drawable is null, the other must be too */ + if ((drawId == None) != (readId == None)) return BadMatch; - } /* ** Lookup old context. If we have one, it must be in a usable state. @@ -608,20 +603,20 @@ DoMakeCurrent(__GLXclientState * cl, return BadAccess; } - assert(drawId != None); - assert(readId != None); + if (drawId) { + drawPriv = __glXGetDrawable(glxc, drawId, client, &status); + if (drawPriv == NULL) + return status; + } - drawPriv = __glXGetDrawable(glxc, drawId, client, &status); - if (drawPriv == NULL) - return status; - - readPriv = __glXGetDrawable(glxc, readId, client, &status); - if (readPriv == NULL) - return status; - - } - else { + if (readId) { + readPriv = __glXGetDrawable(glxc, readId, client, &status); + if (readPriv == NULL) + return status; + } + } else { /* Switching to no context. Ignore new drawable. */ + glxc = 0; drawPriv = 0; readPriv = 0; -- 2.9.3 _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel