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

[Merged by Bors] - Fix unsoundness in Query::for_each_mut #2045

Closed

Conversation

TheRawMeatball
Copy link
Member

No description provided.

@TheRawMeatball TheRawMeatball added C-Bug An unexpected or incorrect behavior A-ECS Entities, components, systems, and events P-High This is particularly urgent, and deserves immediate attention labels Apr 29, 2021
Copy link
Contributor

@bjorn3 bjorn3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Apr 29, 2021
@cart
Copy link
Member

cart commented Apr 29, 2021

🤦

@cart
Copy link
Member

cart commented Apr 29, 2021

bors r+

bors bot pushed a commit that referenced this pull request Apr 29, 2021
@bors bors bot changed the title Fix unsoundness in Query::for_each_mut [Merged by Bors] - Fix unsoundness in Query::for_each_mut Apr 29, 2021
@bors bors bot closed this Apr 29, 2021
NiklasEi pushed a commit to NiklasEi/bevy that referenced this pull request May 1, 2021
ostwilkens pushed a commit to ostwilkens/bevy that referenced this pull request Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ECS Entities, components, systems, and events C-Bug An unexpected or incorrect behavior P-High This is particularly urgent, and deserves immediate attention S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants