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.

Sign up for GitHub

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

Jump to bottom

Web and native backends disagree on some types being Send and Sync #3026

Closed
i509VCB opened this issue Sep 15, 2022 · 2 comments
Closed

Web and native backends disagree on some types being Send and Sync #3026

i509VCB opened this issue Sep 15, 2022 · 2 comments
Labels
area: api Issues related to API surface

Comments

@i509VCB
Copy link
Contributor

i509VCB commented Sep 15, 2022 •
edited
Loading

In #3025 when implementing static assertions to ensure types properly implement (or don't) Send and Sync, I noticed a few types do not agree depending whether you are compiling for web or native.

The following types differ on native and web:

Type Native Web
RenderPass Send + Sync !Send + !Sync
ComputePass Send + Sync !Send + !Sync
BufferView Send + Sync !Send + !Sync
BufferViewMut Send + Sync !Send + !Sync
@i509VCB i509VCB mentioned this issue Sep 16, 2022
Merged
3 tasks
@teoxoy teoxoy added the area: api Issues related to API surface label Dec 5, 2022
@teoxoy
Copy link
Member

teoxoy commented Feb 23, 2023

Related: #2652 & #2884

@daxpedda daxpedda mentioned this issue Apr 15, 2023
Merged
3 tasks
@teoxoy
Copy link
Member

teoxoy commented Jun 30, 2023

Fixed by #3691

@teoxoy teoxoy closed this as completed Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: api Issues related to API surface
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@teoxoy @i509VCB