fix WX_GL_STEREO attribute handling: value for it was missing in the attribute list

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@48536 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/docs/changes.txt b/docs/changes.txt
index e45b89725fe1fa5ddf3f27b46e284534669d6d28..377b4a831b789254359d1861dac330955628919c 100644 (file)
--- a/docs/changes.txt
+++ b/docs/changes.txt
@@ -229,6 +229,7 @@ wxGTK:
 - Add right button event for wxToolbar's tools (Tim Kosse)
 - Don't unconditionally add wxCAPTION style to wxMiniFrame
 - Generate wxEVT_COMMAND_LIST_END_LABEL_EDIT event even if label didn't change
+- Fix WX_GL_STEREO attribute handling (Tristan Mehamli)
 
 
 2.8.4

diff --git a/src/unix/glx11.cpp b/src/unix/glx11.cpp
index ffa31f4eb4b1261e7ad4841c852f5bf950fa8d62..b4a8b2d147c0f4add908ec1cb97ebe37d60233a5 100644 (file)
--- a/src/unix/glx11.cpp
+++ b/src/unix/glx11.cpp
@@ -185,17 +185,21 @@ wxGLCanvasX11::ConvertWXAttrsToGL(const int *wxattrs, int *glattrs, size_t n)
                     glattrs[p++] = GLX_LEVEL;
                     break;
 
+                    // the following boolean attributes don't have values in wx
+                    // API (they're turned on if specified) but do have them in
+                    // OpenGL, so do put them into glattrs and also skip the
+                    // copy of wx value after switch by using "continue"
+                    // instead of "break"
                 case WX_GL_DOUBLEBUFFER:
                     glattrs[p++] = GLX_DOUBLEBUFFER;
                     glattrs[p++] = True;
-
-                    // again, we don't have value for this one in wx list (even
-                    // though OpenGL does use it)
                     continue;
 
                 case WX_GL_STEREO:
                     glattrs[p++] = GLX_STEREO;
-                    break;
+                    glattrs[p++] = True;
+                    continue;
+
 
                 case WX_GL_AUX_BUFFERS:
                     glattrs[p++] = GLX_AUX_BUFFERS;