Skip to content

hal/gles: support externally initialized contexts#2350

Merged
kvark merged 1 commit intogfx-rs:masterfrom
kvark:gles-external
Jan 4, 2022
Merged

hal/gles: support externally initialized contexts#2350
kvark merged 1 commit intogfx-rs:masterfrom
kvark:gles-external

Conversation

@kvark
Copy link
Copy Markdown
Member

@kvark kvark commented Jan 3, 2022

Connections
Closes #2344
cc @lpenguin @caiiiycuk

Description
This PR attempts to make minimal set of changes to be able to use GLES in a context we aren't creating.
Overall, I'm happy with the way it turned out, especially struct EglContext part.

Testing
Comes with a whole example!

@kvark kvark requested a review from cwfitzgerald January 3, 2022 05:57
@caiiiycuk
Copy link
Copy Markdown
Contributor

Have the same error as in #2297 , so problem is not related to winit.

bug.mp4

@caiiiycuk
Copy link
Copy Markdown
Contributor

glutin does not support wasm taget (rust-windowing/glutin#1167), so for wasm need to create separate example in future.

@kvark
Copy link
Copy Markdown
Member Author

kvark commented Jan 3, 2022

Have the same error as in #2297

The examples in wgpu-hal aren't handling window resize, and it's not their goal to do so. This is irrelevant of the PR here.

glutin does not support wasm taget (rust-windowing/glutin#1167), so for wasm need to create separate example in future.

we could put stuff in the same example, just not using glutin, via a bunch of #[cfg()]

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pure GL context

3 participants