From: Jesse Barnes <jbarnes@virtuousgeek.org>
Date: Fri, 5 Mar 2010 19:21:12 +0000 (-0800)
Subject: progs/xdemos: fix up GLX extension checking
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f846a008e9e63f01bc673e31efd0d6b22e4eb0b9;p=mesa.git

progs/xdemos: fix up GLX extension checking

A few of my programs were using the client string in addition to the
main extensions string to see if a feature was present.  Correct this to
only check for the currently available and working extensions.
---

diff --git a/progs/xdemos/glsync.c b/progs/xdemos/glsync.c
index c00ba9e4687..3751373e23b 100644
--- a/progs/xdemos/glsync.c
+++ b/progs/xdemos/glsync.c
@@ -63,19 +63,12 @@ void (*swap_interval)();
 
 static int GLXExtensionSupported(Display *dpy, const char *extension)
 {
-	const char *extensionsString, *client_extensions, *pos;
+	const char *extensionsString, *pos;
 
 	extensionsString = glXQueryExtensionsString(dpy, DefaultScreen(dpy));
-	client_extensions = glXGetClientString(dpy, GLX_EXTENSIONS);
 
 	pos = strstr(extensionsString, extension);
 
-	if (pos != NULL && (pos == extensionsString || pos[-1] == ' ') &&
-	    (pos[strlen(extension)] == ' ' || pos[strlen(extension)] == '\0'))
-		return 1;
-
-	pos = strstr(client_extensions, extension);
-
 	if (pos != NULL && (pos == extensionsString || pos[-1] == ' ') &&
 	    (pos[strlen(extension)] == ' ' || pos[strlen(extension)] == '\0'))
 		return 1;
@@ -235,7 +228,7 @@ int main(int argc, char *argv[])
 
 	XMapWindow(disp, winGL);
 	ret = glXMakeCurrent(disp, winGL, context);
-	if (ret) {
+	if (!ret) {
 		fprintf(stderr, "failed to make context current: %d\n", ret);
 	}
 
diff --git a/progs/xdemos/msctest.c b/progs/xdemos/msctest.c
index 001ecf04d6c..11b04344426 100644
--- a/progs/xdemos/msctest.c
+++ b/progs/xdemos/msctest.c
@@ -45,19 +45,12 @@ void (*wait_sync)(Display *dpy, Window winGL, int64_t target_msc, int64_t diviso
 
 static int GLXExtensionSupported(Display *dpy, const char *extension)
 {
-	const char *extensionsString, *client_extensions, *pos;
+	const char *extensionsString, *pos;
 
 	extensionsString = glXQueryExtensionsString(dpy, DefaultScreen(dpy));
-	client_extensions = glXGetClientString(dpy, GLX_EXTENSIONS);
 
 	pos = strstr(extensionsString, extension);
 
-	if (pos != NULL && (pos == extensionsString || pos[-1] == ' ') &&
-	    (pos[strlen(extension)] == ' ' || pos[strlen(extension)] == '\0'))
-		return 1;
-
-	pos = strstr(client_extensions, extension);
-
 	if (pos != NULL && (pos == extensionsString || pos[-1] == ' ') &&
 	    (pos[strlen(extension)] == ' ' || pos[strlen(extension)] == '\0'))
 		return 1;
@@ -167,8 +160,8 @@ int main(int argc, char *argv[])
 
 	glXMakeCurrent(disp, winGL, context);
 
-	get_sync_values = glXGetProcAddress((unsigned char *)"glXGetSyncValuesOML");
-	wait_sync = glXGetProcAddress((unsigned char *)"glXWaitForMscOML");
+	get_sync_values = (void *)glXGetProcAddress((unsigned char *)"glXGetSyncValuesOML");
+	wait_sync = (void *)glXGetProcAddress((unsigned char *)"glXWaitForMscOML");
 
 	if (!get_sync_values || !wait_sync) {
 		fprintf(stderr, "failed to get sync values function\n");
diff --git a/progs/xdemos/omlsync.c b/progs/xdemos/omlsync.c
index f2c1d13f811..a2baf4ad720 100644
--- a/progs/xdemos/omlsync.c
+++ b/progs/xdemos/omlsync.c
@@ -73,19 +73,12 @@ int (*glXSwapInterval)(int interval);
 
 static int GLXExtensionSupported(Display *dpy, const char *extension)
 {
-	const char *extensionsString, *client_extensions, *pos;
+	const char *extensionsString, *pos;
 
 	extensionsString = glXQueryExtensionsString(dpy, DefaultScreen(dpy));
-	client_extensions = glXGetClientString(dpy, GLX_EXTENSIONS);
 
 	pos = strstr(extensionsString, extension);
 
-	if (pos != NULL && (pos == extensionsString || pos[-1] == ' ') &&
-	    (pos[strlen(extension)] == ' ' || pos[strlen(extension)] == '\0'))
-		return 1;
-
-	pos = strstr(client_extensions, extension);
-
 	if (pos != NULL && (pos == extensionsString || pos[-1] == ' ') &&
 	    (pos[strlen(extension)] == ' ' || pos[strlen(extension)] == '\0'))
 		return 1;