Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prefer double buffer on wgl if unspecified #705

Merged
merged 1 commit into from Jan 14, 2016
Merged

Prefer double buffer on wgl if unspecified #705

merged 1 commit into from Jan 14, 2016

Conversation

dten
Copy link
Contributor

@dten dten commented Jan 14, 2016

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

should this also be done for https://github.com/tomaka/glutin/blob/master/src/api/wgl/mod.rs#L544

@tomaka
Copy link
Contributor

tomaka commented Jan 14, 2016

Yes, please also change line 544. But also add a small comment explaining that in theory we shouldn't be doing that.

The change at line 544 is the most important, as ChoosePixelFormat is only used as a fallback.

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
@dten
Copy link
Contributor Author

dten commented Jan 14, 2016

For me this makes my glium project alive on all platforms again 👍 happy days

tomaka added a commit that referenced this pull request Jan 14, 2016
Prefer double buffer on wgl if unspecified
@tomaka tomaka merged commit 9ba16e3 into rust-windowing:master Jan 14, 2016
@tomaka
Copy link
Contributor

tomaka commented Jan 14, 2016

Note that glium is not supposed to display a black screen for single buffering, I guess it's another bug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants