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

[wasm] Allow enums in unmanaged callbacks. #61377

Merged
merged 1 commit into from
Nov 10, 2021
Merged

Conversation

vargaz
Copy link
Contributor

@vargaz vargaz commented Nov 9, 2021

Fixes #61144.

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@vargaz vargaz requested a review from radical November 9, 2021 17:34
@lambdageek
Copy link
Member

I think we should fix this in 6.0.x servicing @lewing

@radical
Copy link
Member

radical commented Nov 9, 2021

I will try to add tests for this in https://github.com/dotnet/runtime/blob/main/src/tests/BuildWasmApps/Wasm.Build.Tests/PInvokeTableGeneratorTests.cs .

@lewing lewing merged commit a8cc1d0 into dotnet:main Nov 10, 2021
@lewing
Copy link
Member

lewing commented Nov 10, 2021

/backport to release/6.0

@github-actions
Copy link
Contributor

Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/1442281416

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

Successfully merging this pull request may close these issues.

Enums are not considered blittable by Emscripten / Blazor WASM AOT
4 participants