Skip to content

Add "EGL_ANGLE_flexible_surface_compatibility" ext check #13080

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

Merged
merged 3 commits into from
Oct 1, 2023

Conversation

maxkatz6
Copy link
Member

@maxkatz6 maxkatz6 commented Sep 30, 2023

What does the pull request do?

Extension check that fixes Avalonia compatibility with newer Angle versions.
"EGL_ANGLE_flexible_surface_compatibility" extension was removed somewhere after 2020 angle releases, while Avalonia still used it.

How was the solution implemented (if it's not obvious)?

We could remove this attribute and upgrade Angle to the newest, but to keep compatibility with older native binaries, an extension check was added.
Also, as this code is reused between normal Angle and DXGI, some code was reused.

Fixes

Probably fixes #10405 (comment) specific comment. And also possibly fixes Angle on ARM64, when used with newer builds (Edge builds for example).

@maxkatz6 maxkatz6 added the backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch label Sep 30, 2023
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0040215-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@maxkatz6 maxkatz6 force-pushed the check-if-extension-exists branch from f89f070 to 0f4e467 Compare September 30, 2023 08:47
@maxkatz6 maxkatz6 changed the title Abstract WrapDirect3D11Texture into a new class with new Extensions check Add "EGL_ANGLE_flexible_surface_compatibility" ext check Sep 30, 2023
@maxkatz6 maxkatz6 requested a review from kekekeks September 30, 2023 09:21
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0040240-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@maxkatz6 maxkatz6 added this pull request to the merge queue Oct 1, 2023
@maxkatz6 maxkatz6 removed this pull request from the merge queue due to a manual request Oct 1, 2023
@maxkatz6 maxkatz6 merged commit ceab482 into master Oct 1, 2023
@maxkatz6 maxkatz6 deleted the check-if-extension-exists branch October 1, 2023 01:12
grokys pushed a commit that referenced this pull request Oct 2, 2023
* Add "EGL_ANGLE_flexible_surface_compatibility" ext check

* Use stackalloc with CreatePBufferFromClientBuffer attributes

* Fix broken API
@grokys grokys added backported-11.0.x and removed backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch labels Oct 14, 2023
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.

[ARM64][Windows 11]: Running Freshly Created Avalonia 11.0.0-preview4/5 application returns a transparent window
4 participants