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

Always set the swap interval on GLX #832

Closed
wants to merge 1 commit into from

Conversation

jrmuizel
Copy link
Contributor

This matches the behaviour of the WGL backend and allows
toggling vsync in situations where the swap interval defaults
to 1

This matches the behaviour of the WGL backend and allows
toggling vsync in situations where the swap interval defaults
to 1
@tomaka
Copy link
Contributor

tomaka commented Nov 24, 2016

The problem is that "system default" should be a valid option as well.

This has been something that I've wanted to change for some time. In my opinion a better change is that the vsync option should be an enum with three variants (enabled, disabled, system default).

@jrmuizel
Copy link
Contributor Author

So when I first tested this I must have made a mistake because it doesn't actually work. 0 is not a valid value to pass to SwapIntervalSGI and the mesa drivers don't support SwapIntervalExt. The only way to disable vsync with mesa is through SwapIntervalMESA.

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

3 participants