Skip to content

minor *dyn cast cleanup#131898

Merged
bors merged 2 commits intorust-lang:masterfrom
lukas-code:ptr-cast-cleanup
Oct 24, 2024
Merged

minor *dyn cast cleanup#131898
bors merged 2 commits intorust-lang:masterfrom
lukas-code:ptr-cast-cleanup

Conversation

@lukas-code
Copy link
Copy Markdown
Member

Small follow-up to #130234 to remove a redundant check and clean up comments. No functional changes.

Also, explain why casts cannot drop the principal even though coercions can, and add a test because apparently we didn't have one already.

r? @WaffleLapkin or @compiler-errors

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants