Fixed wrong type being passed to eglGetConfigAttrib() (thanks, Alexander!).
authorRyan C. Gordon <icculus@icculus.org>
Fri, 19 Jul 2013 00:22:57 -0400
changeset 7471 75726efbf679
parent 7470 e2c81eca0326
child 7472 0ec51bd57d56
Fixed wrong type being passed to eglGetConfigAttrib() (thanks, Alexander!). This wanted an EGLint*, not a VisualID* cast to an EGLint*. Without this, 64-bit X11 fails here, because the datatype sizes are different. Fixes Bugzilla 1686.
src/video/x11/SDL_x11opengles.c
--- a/src/video/x11/SDL_x11opengles.c	Thu Jul 18 22:20:09 2013 -0400
+++ b/src/video/x11/SDL_x11opengles.c	Fri Jul 19 00:22:57 2013 -0400
@@ -212,7 +212,7 @@
     /* 64 seems nice. */
     EGLint attribs[64];
     EGLint found_configs = 0;
-    VisualID visual_id;
+    EGLint visual_id;
     int i;
 
     if (!_this->gles_data) {
@@ -277,7 +277,7 @@
     if (_this->gles_data->eglGetConfigAttrib(_this->gles_data->egl_display,
                                              _this->gles_data->egl_config,
                                              EGL_NATIVE_VISUAL_ID,
-                                             (EGLint *) & visual_id) ==
+                                             &visual_id) ==
         EGL_FALSE || !visual_id) {
         /* Use the default visual when all else fails */
         XVisualInfo vi_in;