From: John Sheu <sheu@google.com>
Date: Fri, 1 Apr 2016 23:52:22 +0000 (-0700)
Subject: xlib: remove MESA_GLX_VISUAL_HACK
X-Git-Tag: upstream/17.1.0~11023
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f8752e0d95b0b562ca64e8064e98a0b66e5d1591;p=platform%2Fupstream%2Fmesa.git

xlib: remove MESA_GLX_VISUAL_HACK

This removes a hack introduced in 1999 in the first version of
fakeglx.c, with the comment:

  /* XXX revisit this after 3.0 is finished. */

Mesa 4.0 was released in 2001.  It is now 2016, and Mesa 11.0 was
released last year.

Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
---

diff --git a/src/mesa/drivers/x11/fakeglx.c b/src/mesa/drivers/x11/fakeglx.c
index 508dc65..394800f 100644
--- a/src/mesa/drivers/x11/fakeglx.c
+++ b/src/mesa/drivers/x11/fakeglx.c
@@ -283,29 +283,25 @@ save_glx_visual( Display *dpy, XVisualInfo *vinfo,
    if (getenv("MESA_GLX_FORCE_ALPHA"))
       alphaFlag = GL_TRUE;
 
-   /* Comparing IDs uses less memory but sometimes fails. */
-   /* XXX revisit this after 3.0 is finished. */
-   if (!getenv("MESA_GLX_VISUAL_HACK")) {
-      /* First check if a matching visual is already in the list */
-      for (i=0; i<NumVisuals; i++) {
-         XMesaVisual v = VisualTable[i];
-         if (v->display == dpy
-             && v->mesa_visual.level == level
-             && v->mesa_visual.numAuxBuffers == numAuxBuffers
-             && v->ximage_flag == ximageFlag
-             && v->mesa_visual.doubleBufferMode == dbFlag
-             && v->mesa_visual.stereoMode == stereoFlag
-             && (v->mesa_visual.alphaBits > 0) == alphaFlag
-             && (v->mesa_visual.depthBits >= depth_size || depth_size == 0)
-             && (v->mesa_visual.stencilBits >= stencil_size || stencil_size == 0)
-             && (v->mesa_visual.accumRedBits >= accumRedSize || accumRedSize == 0)
-             && (v->mesa_visual.accumGreenBits >= accumGreenSize || accumGreenSize == 0)
-             && (v->mesa_visual.accumBlueBits >= accumBlueSize || accumBlueSize == 0)
-             && (v->mesa_visual.accumAlphaBits >= accumAlphaSize || accumAlphaSize == 0)) {
-            /* now compare visual IDs */
-            if (v->visinfo->visualid == vinfo->visualid) {
-               return v;
-            }
+   /* First check if a matching visual is already in the list */
+   for (i=0; i<NumVisuals; i++) {
+      XMesaVisual v = VisualTable[i];
+      if (v->display == dpy
+          && v->mesa_visual.level == level
+          && v->mesa_visual.numAuxBuffers == numAuxBuffers
+          && v->ximage_flag == ximageFlag
+          && v->mesa_visual.doubleBufferMode == dbFlag
+          && v->mesa_visual.stereoMode == stereoFlag
+          && (v->mesa_visual.alphaBits > 0) == alphaFlag
+          && (v->mesa_visual.depthBits >= depth_size || depth_size == 0)
+          && (v->mesa_visual.stencilBits >= stencil_size || stencil_size == 0)
+          && (v->mesa_visual.accumRedBits >= accumRedSize || accumRedSize == 0)
+          && (v->mesa_visual.accumGreenBits >= accumGreenSize || accumGreenSize == 0)
+          && (v->mesa_visual.accumBlueBits >= accumBlueSize || accumBlueSize == 0)
+          && (v->mesa_visual.accumAlphaBits >= accumAlphaSize || accumAlphaSize == 0)) {
+         /* now compare visual IDs */
+         if (v->visinfo->visualid == vinfo->visualid) {
+            return v;
          }
       }
    }