Fixed issue 9 (update opengl version of a context on OS X)

src/SFML/Window/OSX/SFContext.hpp

@@ -150,6 +150,13 @@ private:
                        unsigned int bitsPerPixel, 
                        const ContextSettings& settings);
     
+    ////////////////////////////////////////////////////////////
+    /// \brief Finish updating our settings
+    /// \note The context must be active in order to get the OpenGL version.
+    ///
+    ////////////////////////////////////////////////////////////
+    void UpdateOpenGLVersion();
+    
     ////////////////////////////////////////////////////////////
     // Member data
     ////////////////////////////////////////////////////////////

src/SFML/Window/OSX/SFContext.mm

@@ -60,6 +60,9 @@ SFContext::SFContext(SFContext* shared)
     
     // Activate the context
     SetActive(true);
+    
+    // Finish updating settings.
+    UpdateOpenGLVersion();
 }
 
     
@@ -79,6 +82,9 @@ SFContext::SFContext(SFContext* shared, const ContextSettings& settings,
     
     // Activate the context
     SetActive(true);
+    
+    // Finish updating settings.
+    UpdateOpenGLVersion();
 }
 
 
@@ -106,8 +112,12 @@ SFContext::SFContext(SFContext* shared, const ContextSettings& settings,
     
     // Activate the context
     SetActive(true);
+    
+    // Finish updating settings.
+    UpdateOpenGLVersion();
 }
 
+
 ////////////////////////////////////////////////////////////
 SFContext::~SFContext()
 {
@@ -210,12 +220,37 @@ void SFContext::CreateContext(SFContext* shared,
     // Free up.
     [pixFmt release];
     
-#warning update settings with ogl version not yet implemented
+    // Save the settings. (OpenGL version is updated elsewhere.)
-    
-    // Save the creation settings
     mySettings = settings;
 }
 
+
+////////////////////////////////////////////////////////////
+void SFContext::UpdateOpenGLVersion()
+{
+    // Update the OpenGL version in the settings.
+    // NB : the context muste be active to get its version.
+
+    GLubyte const* versionString = glGetString(GL_VERSION);
+    
+    if (versionString == 0) {
+
+        // (Warning) Couldn't get the OpenGL version of the context.
+        // This happens sometimes (but not always) when creating the first
+        // context for no apparent reason.
+        
+        // We assume we can get at least a 2.0 valid context.
+        mySettings.MajorVersion = 2;
+        mySettings.MinorVersion = 0;
+        
+    } else {
+        
+        // versionString looks like "2.1 NVIDIA-1.6.26".
+        mySettings.MajorVersion = versionString[0];
+        mySettings.MinorVersion = versionString[2];
+    }
+}
+
 } // namespace priv
 
 } // namespace sf