Mesa (master): xdemos/glsync: check glXMakeCurrent return value

Jesse Barnes jbarnes at kemper.freedesktop.org
Mon Jan 11 20:24:17 UTC 2010


Module: Mesa
Branch: master
Commit: 0269dc19b57b61d46b09fe2636ce430baa9383c3
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0269dc19b57b61d46b09fe2636ce430baa9383c3

Author: Jesse Barnes <jbarnes at virtuousgeek.org>
Date:   Mon Jan  4 13:13:57 2010 -0500

xdemos/glsync: check glXMakeCurrent return value

Since this program is used for testing, catching this case can be helpful.

---

 progs/xdemos/glsync.c |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/progs/xdemos/glsync.c b/progs/xdemos/glsync.c
index 0608f75..6108d99 100644
--- a/progs/xdemos/glsync.c
+++ b/progs/xdemos/glsync.c
@@ -120,6 +120,7 @@ int main(int argc, char *argv[])
 	int width = 500, height = 500, verbose = 0,
 		countonly = 0, interval = 1;
 	int c, i = 1;
+	int ret;
 	int attribs[] = { GLX_RGBA,
                      GLX_RED_SIZE, 1,
                      GLX_GREEN_SIZE, 1,
@@ -234,7 +235,10 @@ int main(int argc, char *argv[])
 	}
 
 	XMapWindow(disp, winGL);
-	glXMakeCurrent(disp, winGL, context);
+	ret = glXMakeCurrent(disp, winGL, context);
+	if (ret) {
+		fprintf(stderr, "failed to make context current: %d\n", ret);
+	}
 
 	video_sync_get = glXGetProcAddress((unsigned char *)"glXGetVideoSyncSGI");
 	video_sync = glXGetProcAddress((unsigned char *)"glXWaitVideoSyncSGI");




More information about the mesa-commit mailing list