Update for changes in gl-rs

4 files changed, 62 insertions, 40 deletions

diff --git a/examples/support/mod.rs b/examples/support/mod.rs
index 1e61165..eb27d23 100644
--- a/examples/support/mod.rs
+++ b/examples/support/mod.rs
@@ -31,7 +31,7 @@ pub struct Context {
 }
 
 pub fn load(window: &glutin::Window) -> Context {
-    let gl = gl::Gl::load_with(|symbol| window.get_proc_address(symbol));
+    let gl = gl::Gl::load(window);
 
     let version = {
         use std::c_str::CString;
@@ -46,42 +46,46 @@ pub fn load(window: &glutin::Window) -> Context {
 impl Context {
     #[cfg(not(target_os = "android"))]
     pub fn draw_frame(&self, color: (f32, f32, f32, f32)) {
-        self.gl.ClearColor(color.0, color.1, color.2, color.3);
-        self.gl.Clear(gl::COLOR_BUFFER_BIT);
-
-        self.gl.Begin(gl::TRIANGLES);
-        self.gl.Color3f(1.0, 0.0, 0.0);
-        self.gl.Vertex2f(-0.5, -0.5);
-        self.gl.Color3f(0.0, 1.0, 0.0);
-        self.gl.Vertex2f(0.0, 0.5);
-        self.gl.Color3f(0.0, 0.0, 1.0);
-        self.gl.Vertex2f(0.5, -0.5);
-        self.gl.End();
-
-        self.gl.Flush();
+        unsafe {
+            self.gl.ClearColor(color.0, color.1, color.2, color.3);
+            self.gl.Clear(gl::COLOR_BUFFER_BIT);
+
+            self.gl.Begin(gl::TRIANGLES);
+            self.gl.Color3f(1.0, 0.0, 0.0);
+            self.gl.Vertex2f(-0.5, -0.5);
+            self.gl.Color3f(0.0, 1.0, 0.0);
+            self.gl.Vertex2f(0.0, 0.5);
+            self.gl.Color3f(0.0, 0.0, 1.0);
+            self.gl.Vertex2f(0.5, -0.5);
+            self.gl.End();
+
+            self.gl.Flush();
+        }
     }
 
     #[cfg(target_os = "android")]
     pub fn draw_frame(&self, color: (f32, f32, f32, f32)) {
-        self.gl.ClearColor(color.0, color.1, color.2, color.3);
-        self.gl.Clear(gl::COLOR_BUFFER_BIT);
-
-        self.gl.EnableClientState(gl::VERTEX_ARRAY);
-        self.gl.EnableClientState(gl::COLOR_ARRAY);
-
         unsafe {
-            use std::mem;
-            self.gl.VertexPointer(2, gl::FLOAT, (mem::size_of::<f32>() * 5) as i32,
-                mem::transmute(VERTEX_DATA.as_slice().as_ptr()));
-            self.gl.ColorPointer(3, gl::FLOAT, (mem::size_of::<f32>() * 5) as i32,
-                mem::transmute(VERTEX_DATA.as_slice().as_ptr().offset(2)));
+            self.gl.ClearColor(color.0, color.1, color.2, color.3);
+            self.gl.Clear(gl::COLOR_BUFFER_BIT);
+
+            self.gl.EnableClientState(gl::VERTEX_ARRAY);
+            self.gl.EnableClientState(gl::COLOR_ARRAY);
+
+            unsafe {
+                use std::mem;
+                self.gl.VertexPointer(2, gl::FLOAT, (mem::size_of::<f32>() * 5) as i32,
+                    mem::transmute(VERTEX_DATA.as_slice().as_ptr()));
+                self.gl.ColorPointer(3, gl::FLOAT, (mem::size_of::<f32>() * 5) as i32,
+                    mem::transmute(VERTEX_DATA.as_slice().as_ptr().offset(2)));
+            }
+
+            self.gl.DrawArrays(gl::TRIANGLES, 0, 3);
+            self.gl.DisableClientState(gl::VERTEX_ARRAY);
+            self.gl.DisableClientState(gl::COLOR_ARRAY);
+            
+            self.gl.Flush();
         }
-
-        self.gl.DrawArrays(gl::TRIANGLES, 0, 3);
-        self.gl.DisableClientState(gl::VERTEX_ARRAY);
-        self.gl.DisableClientState(gl::COLOR_ARRAY);
-        
-        self.gl.Flush();
     }
 }
 
diff --git a/src/lib.rs b/src/lib.rs
index f1f8d5a..a976da5 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -31,6 +31,8 @@
 
 #[phase(plugin)] extern crate compile_msg;
 #[phase(plugin)] extern crate gl_generator;
+
+extern crate gl_common;
 extern crate libc;
 
 #[cfg(target_os = "macos")]
@@ -397,6 +399,13 @@ impl Window {
     }
 }
 
+#[cfg(feature = "window")]
+impl gl_common::GlFunctionsSource for Window {
+    fn get_proc_addr(&self, addr: &str) -> *const libc::c_void {
+        self.get_proc_address(addr)
+    }
+}
+
 /// Represents a headless OpenGL context.
 #[cfg(feature = "headless")]
 pub struct HeadlessContext {
@@ -421,6 +430,13 @@ impl HeadlessContext {
     }
 }
 
+#[cfg(feature = "headless")]
+impl gl_common::GlFunctionsSource for HeadlessContext {
+    fn get_proc_addr(&self, addr: &str) -> *const libc::c_void {
+        self.get_proc_address(addr)
+    }
+}
+
 /// An iterator for the `poll_events` function.
 // Implementation note: we retreive the list once, then serve each element by one by one.
 // This may change in the future.
diff --git a/src/win32/init.rs b/src/win32/init.rs
index bf84973..73a6a6a 100644
--- a/src/win32/init.rs
+++ b/src/win32/init.rs
@@ -350,7 +350,7 @@ pub fn new_window(builder_dimensions: Option<(uint, uint)>, builder_title: Strin
         if builder_vsync {
             if extra_functions.SwapIntervalEXT.is_loaded() {
                 unsafe { ffi::wgl::MakeCurrent(hdc, context) };
-                if extra_functions.SwapIntervalEXT(1) == 0 {
+                if unsafe { extra_functions.SwapIntervalEXT(1) } == 0 {
                     tx.send(Err(format!("wglSwapIntervalEXT failed")));
                     unsafe { ffi::wgl::DeleteContext(context); }
                     unsafe { ffi::DestroyWindow(real_window); }
diff --git a/tests/headless.rs b/tests/headless.rs
index 3c587ea..fe82c1d 100644
--- a/tests/headless.rs
+++ b/tests/headless.rs
@@ -24,12 +24,14 @@ fn main() {
 
     let gl = gl::Gl::load_with(|symbol| window.get_proc_address(symbol));
 
-    gl.ClearColor(0.0, 1.0, 0.0, 1.0);
-    gl.Clear(gl::COLOR_BUFFER_BIT);
-
-    let mut value: (u8, u8, u8, u8) = unsafe { std::mem::uninitialized() };
-    unsafe { gl.ReadPixels(0, 0, 1, 1, gl::RGBA, gl::UNSIGNED_BYTE, std::mem::transmute(&mut value)) };
-
-    assert!(value == (0, 255, 0, 255) || value == (0, 64, 0, 255) ||
-        value == (0, 64, 0, 255) || value == (0, 64, 0, 0));
+    unsafe {
+        gl.ClearColor(0.0, 1.0, 0.0, 1.0);
+        gl.Clear(gl::COLOR_BUFFER_BIT);
+
+        let mut value: (u8, u8, u8, u8) = std::mem::uninitialized();
+        gl.ReadPixels(0, 0, 1, 1, gl::RGBA, gl::UNSIGNED_BYTE, std::mem::transmute(&mut value));
+        
+        assert!(value == (0, 255, 0, 255) || value == (0, 64, 0, 255) ||
+            value == (0, 64, 0, 255) || value == (0, 64, 0, 0));
+    }
 }