egl: remove Options::Platform handling
authorAdam Jackson <ajax@redhat.com>
Thu, 16 May 2019 17:01:36 +0000 (18:01 +0100)
committerMarek Olšák <marek.olsak@amd.com>
Wed, 5 Jun 2019 17:35:21 +0000 (13:35 -0400)
The full set of attributes is already handled with previous patches.
Thus all this is not dead code.

v2 (Emil) - split from a larger patch.

Reviewed-by: Mathias Fröhlich <Mathias.Froehlich@web.de>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Signed-off-by: Marek Olšák <marek.olsak@amd.com>
src/egl/main/egldisplay.c
src/egl/main/egldisplay.h

index 9a2ac48e8bc16d8b63371486a610435d65c3a9b5..418ab0ec9b871a12a80ee212cf9e44752141aeb9 100644 (file)
@@ -497,17 +497,12 @@ _eglParseX11DisplayAttribList(_EGLDisplay *display,
       return EGL_TRUE;
    }
 
       return EGL_TRUE;
    }
 
+   /* EGL_EXT_platform_x11 recognizes exactly one attribute,
+    * EGL_PLATFORM_X11_SCREEN_EXT, which is optional.
+    */
    for (i = 0; attrib_list[i] != EGL_NONE; i += 2) {
    for (i = 0; attrib_list[i] != EGL_NONE; i += 2) {
-      EGLAttrib attrib = attrib_list[i];
-      EGLAttrib value = attrib_list[i + 1];
-
-      /* EGL_EXT_platform_x11 recognizes exactly one attribute,
-       * EGL_PLATFORM_X11_SCREEN_EXT, which is optional.
-       */
-      if (attrib != EGL_PLATFORM_X11_SCREEN_EXT)
+      if (attrib_list[i] != EGL_PLATFORM_X11_SCREEN_EXT)
          return _eglError(EGL_BAD_ATTRIBUTE, "eglGetPlatformDisplay");
          return _eglError(EGL_BAD_ATTRIBUTE, "eglGetPlatformDisplay");
-
-      display->Options.Platform = (void *)(uintptr_t)value;
    }
 
    return EGL_TRUE;
    }
 
    return EGL_TRUE;
index 2c9fd6c339946f572d8a869b99e315fceb545905..369ab31187f7b1c02e4f7c5ae2805f48c9e2a029 100644 (file)
@@ -167,7 +167,6 @@ struct _egl_display
    /* options that affect how the driver initializes the display */
    struct {
       EGLBoolean ForceSoftware; /**< Use software path only */
    /* options that affect how the driver initializes the display */
    struct {
       EGLBoolean ForceSoftware; /**< Use software path only */
-      void *Platform;         /**< Platform-specific options */
       EGLAttrib *Attribs;     /**< Platform-specific options */
    } Options;
 
       EGLAttrib *Attribs;     /**< Platform-specific options */
    } Options;