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
[GL] Android emulator support #3592
Conversation
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.
Thank you for the PR!
a54390c
to
0ae506b
Compare
@kvark I think I fixed all suggestions. Thank you for reviewing! |
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.
Looks wonderful, just one small nit left!
src/backend/gl/src/window/egl.rs
Outdated
.make_current(inner.display, None, None, Some(inner.context)) | ||
.unwrap(); | ||
if let Some(pbuffer) = inner.pbuffer { | ||
inner |
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.
do we actually need this if
? couldn't we just pass make_current(inner.display, inner.pbuffer, inner.pbuffer, ..)
right away?
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.
Removed if
and checked on the emulator and device. Everything is working fine.
0ae506b
to
41d2c5f
Compare
Sweet!
Bors r+
… On Jan 16, 2021, at 15:32, Igor Shaposhnik ***@***.***> wrote:
@Gordon-F commented on this pull request.
In src/backend/gl/src/window/egl.rs:
> @@ -285,10 +298,23 @@ impl hal::Instance<crate::Backend> for Instance {
fn enumerate_adapters(&self) -> Vec<hal::adapter::Adapter<crate::Backend>> {
let inner = self.inner.lock();
- inner
- .egl
- .make_current(inner.display, None, None, Some(inner.context))
- .unwrap();
+ if let Some(pbuffer) = inner.pbuffer {
+ inner
Removed if and checked on the emulator and device. Everything is working fine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
PR checklist:
Android emulator doesn't support EGL_KHR_surfaceless_context.