From 6c73209746331f1fe3bac402b58c1702798f7e62 Mon Sep 17 00:00:00 2001 From: Zetok Zalbavar Date: Fri, 14 Aug 2015 14:56:17 +0100 Subject: Prefer GLX over EGL fixes #314 for me. I've "tested" change by running examples (which prior to change simply crashed), but since I did not run those examples successfuly ever before, I don't know whether they worked as intended. --- src/api/x11/window.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'src/api/x11/window.rs') diff --git a/src/api/x11/window.rs b/src/api/x11/window.rs index f6f944d..36b60e4 100644 --- a/src/api/x11/window.rs +++ b/src/api/x11/window.rs @@ -351,10 +351,12 @@ impl Window { let builder_clone = builder.clone(); let context = match builder.gl_version { GlRequest::Latest | GlRequest::Specific(Api::OpenGl, _) | GlRequest::GlThenGles { .. } => { - if let Some(ref egl) = display.egl { - Prototype::Egl(try!(EglContext::new(egl.clone(), &builder_clone, egl::NativeDisplay::X11(Some(display.display as *const _))))) - } else if let Some(ref glx) = display.glx { + // GLX should be preferred over EGL, otherwise crashes may occur + // on X11 – issue #314 + if let Some(ref glx) = display.glx { Prototype::Glx(try!(GlxContext::new(glx.clone(), &display.xlib, &builder_clone, display.display))) + } else if let Some(ref egl) = display.egl { + Prototype::Egl(try!(EglContext::new(egl.clone(), &builder_clone, egl::NativeDisplay::X11(Some(display.display as *const _))))) } else { return Err(CreationError::NotSupported); } -- cgit v1.2.3