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

Regenerate GL loader code with GLAD 2 #68372

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Conversation

Riteo
Copy link
Contributor

@Riteo Riteo commented Nov 7, 2022

GLAD 1 creates unusable loaders for EGL, while the newly released GLAD 2 does not, so for consistency I thought that it would be a good idea to uniform things beforehand. While it had some API changes some renames were all that was needed and everything works like before, at least on the Wayland branch.

I've kept the structure identical, although this new generator has quite a few hefty features, such as a single header mode.

I've also added GLAD to thirdparty/README.md, but I haven't specified that in the commit title because it's a very small "fix".

This patch paves the way for dynamic EGL/GLX loading (very, very preliminarily, but trust me, it does), allowing for good Wayland/X11 interoperablity.

@Riteo Riteo marked this pull request as ready for review November 7, 2022 13:42
@Riteo Riteo requested review from a team as code owners November 7, 2022 13:42
@Geometror Geometror modified the milestones: 4.x, 4.0 Nov 7, 2022
@Riteo Riteo force-pushed the glad2 branch 3 times, most recently from 9fa8a46 to f6397bf Compare November 14, 2022 20:47
GLAD 1 creates unusable loaders for EGL, while the newly released GLAD 2
does not, so for consistency I thought that it would be a good idea to
uniform things beforehand. While it had some API changes some renames
were all that was needed and everything works like before, at least on
the Wayland branch.

I've kept the structure identical, although this new generator has quite
a few hefty features, such as a single header mode.

I've also added GLAD to `thirdparty/README.md`, but I haven't specified
that in the commit title because it's a very small "fix".
Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed as part of #68586, this looks great.

@akien-mga akien-mga merged commit dbf6ff0 into godotengine:master Nov 15, 2022
@akien-mga
Copy link
Member

Thanks!

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

Successfully merging this pull request may close these issues.

3 participants