Skip to content

feat: Support impl and mut restrictions#22022

Merged
ChayimFriedman2 merged 1 commit intorust-lang:masterfrom
ChayimFriedman2:restrictions
Apr 13, 2026
Merged

feat: Support impl and mut restrictions#22022
ChayimFriedman2 merged 1 commit intorust-lang:masterfrom
ChayimFriedman2:restrictions

Conversation

@ChayimFriedman2
Copy link
Copy Markdown
Contributor

I chose the order of mut restrictions to be unsafe mut(...) for unsafe fields, but it's undecided yet.

Closes #22021.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 12, 2026
Copy link
Copy Markdown
Member

@Veykril Veykril left a comment

Choose a reason for hiding this comment

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

We could try to accept both orderings for now (and maybe even retain that for recovery reasons, just emitting a soft error then)?

View changes since this review

@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Apr 13, 2026
@ChayimFriedman2 ChayimFriedman2 removed this pull request from the merge queue due to a manual request Apr 13, 2026
I chose the order of mut restrictions to be `unsafe mut(...)` for unsafe fields, but it's undecided yet.
@ChayimFriedman2 ChayimFriedman2 added this pull request to the merge queue Apr 13, 2026
Merged via the queue into rust-lang:master with commit 545134a Apr 13, 2026
17 checks passed
@ChayimFriedman2 ChayimFriedman2 deleted the restrictions branch April 13, 2026 10:39
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support impl restriction syntax

3 participants