Mesa (9.2): glx: Generate fewer errors in MakeContextCurrent
Carl Worth
cworth at kemper.freedesktop.org
Wed Oct 16 22:10:51 UTC 2013
Module: Mesa
Branch: 9.2
Commit: fb3e55f898e047ed32008e8a579146090cc8f805
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=fb3e55f898e047ed32008e8a579146090cc8f805
Author: Adam Jackson <ajax at redhat.com>
Date: Fri Oct 4 09:25:51 2013 -0400
glx: Generate fewer errors in MakeContextCurrent
For a few reasons.
1: In the (current) common case, these conditionals are never true. All
we're doing by checking them is slowing down MakeCurrent. The server
does these checks already anyway.
2: GLX >= 3.0 contexts may legally be made current without a bound
framebuffer.
This does not fix piglit/glx-create-context-current-no-framebuffer, but
is a prerequisite for fixing it.
Cc: "9.1 9.2" <mesa-stable at lists.freedesktop.org>
Reviewed-by: Brian Paul <brianp at vmware.com>
Signed-off-by: Adam Jackson <ajax at redhat.com>
(cherry picked from commit e166a58c43e92f10f03b32747bff44708096f898)
---
src/glx/glxcurrent.c | 10 ----------
1 files changed, 0 insertions(+), 10 deletions(-)
diff --git a/src/glx/glxcurrent.c b/src/glx/glxcurrent.c
index 3d8893c..a6884cf 100644
--- a/src/glx/glxcurrent.c
+++ b/src/glx/glxcurrent.c
@@ -231,16 +231,6 @@ MakeContextCurrent(Display * dpy, GLXDrawable draw,
return GL_FALSE;
}
- if (gc == NULL && (draw != None || read != None)) {
- __glXGenerateError(dpy, (draw != None) ? draw : read,
- BadMatch, X_GLXMakeContextCurrent);
- return False;
- }
- if (gc != NULL && (draw == None || read == None)) {
- __glXGenerateError(dpy, None, BadMatch, X_GLXMakeContextCurrent);
- return False;
- }
-
_glapi_check_multithread();
__glXLock();
More information about the mesa-commit
mailing list