-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix fp in borrow_interior_mutable_const
#5949
Fix fp in borrow_interior_mutable_const
#5949
Conversation
Fix false positive when referencing a field behind a pointer.
r? @yaahc (rust_highfive has picked a reviewer for you, use r? to override) |
Co-authored-by: Thibaud <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me but I'm not confident I'm wrapping my head around all the ways references and consts interact so I'm going to defer to the rest of the clippy team on approval
cc @rust-lang/clippy
r? @oli-obk on the "referecnes and consts" bit? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting for @oli-obk
@bors r+ yea this makes sense. The only thing I'm wondering is whether we should move this lint to a MIR lint where I think we may be able to write a guaranteed-false-positive-free version since there'd be no more implicit things happening |
📌 Commit ce8915c has been approved by |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
fixes #5796
changelog: fix false positive in
borrow_interior_mutable_const
when referencing a field behind a pointer.