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

EGL and Gles2 cleanups and additions #460

Merged
merged 8 commits into from
Jan 14, 2022
Merged

EGL and Gles2 cleanups and additions #460

merged 8 commits into from
Jan 14, 2022

Conversation

Drakulix
Copy link
Member

@Drakulix Drakulix commented Jan 10, 2022

Best reviewed on a commit-by-commit basis.

Addresses some short-comings I found implementing smithay-egui.

6e1f6ab makes it possible to attach user_data to an EGLContext. Given that GL-resources like textures or buffers are local to a context, this is the obvious place to store them.

e0ad187 makes it possible to retrieve the EGLContext of a Gles2Renderer to actually access the user_data from downstream.

7ac8872 and 07bf1e5 expose rendering specifics of the Gles2Frame, that make it easier to correctly orientate and size externally drawn content.

04f6434 allows downstream to retrieve the raw-egl handles, if they want to. We already supported that for the EGLDevice and EGLDisplay anyway, and this is useful for example to hook up RenderDoc.

08df10c fixes up the existing texture filtering code of the Gles2Renderer, which was previously not applied correctly.

29b0b2b fixes resizing the window of the winit-backend in anvil, which broke with #423.

Finally 864b379 adds a utility function to transform Rectangles and some tests.

@Drakulix Drakulix force-pushed the feature/egl_userdata branch from 1edff97 to 5cbd435 Compare January 12, 2022 21:15
@Drakulix Drakulix merged commit d554c7d into master Jan 14, 2022
@Drakulix Drakulix deleted the feature/egl_userdata branch June 23, 2022 19:34
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.

2 participants