Fixed regression causing the renderer to recreate its window since it's not getting an OpenGL 2.1 context anymore.
authorSam Lantinga <slouken@libsdl.org>
Mon, 10 Mar 2014 18:45:07 -0700
changeset 8601 4c04c7785cba
parent 8600 092802455aed
child 8602 52ad5ab6eb2d
Fixed regression causing the renderer to recreate its window since it's not getting an OpenGL 2.1 context anymore.
src/video/SDL_video.c
src/video/cocoa/SDL_cocoaopengl.m
--- a/src/video/SDL_video.c	Mon Mar 10 21:21:35 2014 -0400
+++ b/src/video/SDL_video.c	Mon Mar 10 18:45:07 2014 -0700
@@ -2563,10 +2563,10 @@
     _this->gl_config.multisamplesamples = 0;
     _this->gl_config.retained_backing = 1;
     _this->gl_config.accelerated = -1;  /* accelerated or not, both are fine */
+    _this->gl_config.profile_mask = 0;
 #if SDL_VIDEO_OPENGL
-    _this->gl_config.major_version = 1;
-    _this->gl_config.minor_version = 2;
-    _this->gl_config.profile_mask = 0;
+    _this->gl_config.major_version = 2;
+    _this->gl_config.minor_version = 1;
 #elif SDL_VIDEO_OPENGL_ES2
     _this->gl_config.major_version = 2;
     _this->gl_config.minor_version = 0;
--- a/src/video/cocoa/SDL_cocoaopengl.m	Mon Mar 10 21:21:35 2014 -0400
+++ b/src/video/cocoa/SDL_cocoaopengl.m	Mon Mar 10 18:45:07 2014 -0700
@@ -162,7 +162,9 @@
 Cocoa_GL_CreateContext(_THIS, SDL_Window * window)
 {
     SDL_VideoData *data = (SDL_VideoData *) _this->driverdata;
+/*
     const GLubyte *(APIENTRY * glGetStringFunc)(GLenum) = NULL;
+*/
     NSAutoreleasePool *pool;
     SDL_VideoDisplay *display = SDL_GetDisplayForWindow(window);
     SDL_DisplayData *displaydata = (SDL_DisplayData *)display->driverdata;
@@ -249,7 +251,7 @@
 
     fmt = [[NSOpenGLPixelFormat alloc] initWithAttributes:attr];
     if (fmt == nil) {
-        SDL_SetError ("Failed creating OpenGL pixel format");
+        SDL_SetError("Failed creating OpenGL pixel format");
         [pool release];
         return NULL;
     }
@@ -263,7 +265,7 @@
     [fmt release];
 
     if (context == nil) {
-        SDL_SetError ("Failed creating OpenGL context");
+        SDL_SetError("Failed creating OpenGL context");
         [pool release];
         return NULL;
     }
@@ -272,10 +274,22 @@
 
     if ( Cocoa_GL_MakeCurrent(_this, window, context) < 0 ) {
         Cocoa_GL_DeleteContext(_this, context);
-        SDL_SetError ("Failed making OpenGL context current");
+        SDL_SetError("Failed making OpenGL context current");
         return NULL;
     }
 
+/* No other backend does this version checking.
+   If we enable it, we should consider whether it should be done at a 
+   higher level for all platforms. We'll have to think through the implications
+   of this.
+
+   For example, Mac OS X 10.6 will only report OpenGL 2.0, but we ask for 2.1
+   by default. If we don't get 2.1, then the renderer will set the requested
+   version and try to recreate the window, which causes all kinds of problems.
+
+   For now, we'll just disable this code until we can think about it more.
+*/
+#if 0
     glGetStringFunc = (const GLubyte *(APIENTRY *)(GLenum)) SDL_GL_GetProcAddress("glGetString");
     if (!glGetStringFunc) {
         Cocoa_GL_DeleteContext(_this, context);
@@ -305,6 +319,7 @@
 
     _this->gl_config.major_version = glversion_major;
     _this->gl_config.minor_version = glversion_minor;
+#endif
 
     return context;
 }