st/xlib: Do minimal version checking in glXCreateContextAttribsARB.
authorJosé Fonseca <jfonseca@vmware.com>
Thu, 24 Apr 2014 14:49:01 +0000 (15:49 +0100)
committerJosé Fonseca <jfonseca@vmware.com>
Thu, 24 Apr 2014 19:26:23 +0000 (20:26 +0100)
The current version checking is wrongly refusing to create 3.3 contexts;
unsupported version are checked elsewhere; and the DRI path doesn't do
this sort of checking neither.

This enables piglit glsl 3.30 tests to run without skipping.

Reviewed-by: Brian Paul <brianp@vmware.com>
src/gallium/state_trackers/glx/xlib/glx_api.c

index 05880ddcdda1335ce0512ad6403c17b08802fef0..d97cbd7d996a6d3e900c49322987ee13089ecafb 100644 (file)
@@ -2724,26 +2724,8 @@ glXCreateContextAttribsARB(Display *dpy, GLXFBConfig config,
    }
 
    /* check version (generate BadMatch if bad) */
-   switch (majorVersion) {
-   case 1:
-      if (minorVersion < 0 || minorVersion > 5)
-         return NULL;
-      break;
-   case 2:
-      if (minorVersion < 0 || minorVersion > 1)
-         return NULL;
-      break;
-   case 3:
-      if (minorVersion < 0 || minorVersion > 2)
-         return NULL;
-      break;
-   case 4:
-      if (minorVersion < 0 || minorVersion > 0)
-         return NULL;
-      break;
-   default:
+   if (majorVersion < 0 || minorVersion < 0)
       return NULL;
-   }
 
    if ((contextFlags & GLX_CONTEXT_FORWARD_COMPATIBLE_BIT_ARB) &&
        majorVersion < 3)