Prefer double buffer on wgl if unspecified

According to https://msdn.microsoft.com/en-us/library/windows/desktop/dd318284(v=vs.85).aspx if double buffer is unavailable then a single buffer will be returned
This commit is contained in:
David Hewson 2016-01-14 15:23:39 +00:00
parent 88e64a87a9
commit 8a7edc4e40

View file

@ -383,7 +383,7 @@ unsafe fn choose_native_pixel_format(hdc: winapi::HDC, reqs: &PixelFormatRequire
nVersion: 1, nVersion: 1,
dwFlags: { dwFlags: {
let f1 = match reqs.double_buffer { let f1 = match reqs.double_buffer {
None => winapi::PFD_DOUBLEBUFFER_DONTCARE, None => winapi::PFD_DOUBLEBUFFER, // Should be PFD_DOUBLEBUFFER_DONTCARE after you can choose
Some(true) => winapi::PFD_DOUBLEBUFFER, Some(true) => winapi::PFD_DOUBLEBUFFER,
Some(false) => 0, Some(false) => 0,
}; };
@ -541,10 +541,10 @@ unsafe fn choose_arb_pixel_format(extra: &gl::wgl_extra::Wgl, extensions: &str,
out.push(stencil as c_int); out.push(stencil as c_int);
} }
if let Some(double_buffer) = reqs.double_buffer { // Prefer double buffering if unspecified (probably shouldn't once you can choose)
out.push(gl::wgl_extra::DOUBLE_BUFFER_ARB as c_int); let double_buffer = reqs.double_buffer.unwrap_or(true);
out.push(if double_buffer { 1 } else { 0 }); out.push(gl::wgl_extra::DOUBLE_BUFFER_ARB as c_int);
} out.push(if double_buffer { 1 } else { 0 });
if let Some(multisampling) = reqs.multisampling { if let Some(multisampling) = reqs.multisampling {
if extensions.split(' ').find(|&i| i == "WGL_ARB_multisample").is_some() { if extensions.split(' ').find(|&i| i == "WGL_ARB_multisample").is_some() {