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
Add API to let users pick OpenGL version #921
Conversation
src/Windowing/OpenToolkit.Windowing.Common/Enums/OpenGLVersion.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mentioned this in Discord, but thought I'd mention it here.
Don't forget we changed the way we bake profiles in 4.0.
- OpenToolkit.OpenGL contains OpenGL 1.0 through OpenGL 4.6
- OpenToolkit.OpenGL.Core contains OpenGL 1.0 through OpenGL 4.6, core profile
- OpenToolkit.OpenGLES contains OpenGLES 1.0 through OpenGLES 3.2
Obsolete functions are marked as such, except in core profile where obsolete functions are removed. This GLVersion would be used for windowing only, and no way affect the bindings.
src/Windowing/OpenToolkit.Windowing.Common/Enums/OpenGLVersion.cs
Outdated
Show resolved
Hide resolved
This is ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, Void. I have a few comments, nothing too major.
src/Windowing/OpenToolkit.Windowing.Common/Enums/ContextFlags.cs
Outdated
Show resolved
Hide resolved
Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome stuff Void, we're getting close now.
Done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great, almost there. Just a a few tiny changes!
Finished. |
Purpose of this PR
Testing status
Comments