-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
as_deref()
and as_deref_mut()
impls
#59628
Conversation
r? @Kimundi (rust_highfive has picked a reviewer for you, use r? to override) |
LGTM, though I'm not a libs reviewer. |
@bors: r+ Seems like a sensible change, based on the discussion so far. |
📌 Commit e71e71b has been approved by |
`as_deref()` and `as_deref_mut()` impls addresses rust-lang#50264 renamed `deref()` -> `as_deref()` added `deref_mut()` impls + tests fixed breaking changes
@bors r- Failed in #59692 (comment) due to miri toolstate. |
Looks like the old Or land this separately from a Miri update so that breaking Miir is not fatal. |
☔ The latest upstream changes (presumably #59910) made this pull request unmergeable. Please resolve the merge conflicts. |
Hi, @RalfJung, I'm not fluent with the Rust release process, but can a feature which has never been stable be deprecated? Or are you recommending the method name be reverted from |
Yes. Just add a |
@U007D We can break miri again so you don't need to use |
Sure, that also works. No idea how widely used this function is. (What happened last time was not about tools not breaking due to beta; it was that this one got rolled up with an "update miri" PR. Changing a tool and breaking it in the same PR leads to CI failure.) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
☔ The latest upstream changes (presumably #58702) made this pull request unmergeable. Please resolve the merge conflicts. |
ping from triage @U007D any updates? |
Triage ping @U007D, this needs a few merge conflicts to be resolved |
ping from triage @U007D |
Introduce `as_deref` to Option This is re-submission for rust-lang#59628. Renames `deref()` to `as_deref()` and adds `deref_mut()` impls and tests. CC rust-lang#50264 r? @Kimundi (I picked you as you're the previous reviewer.)
Introduce `as_deref` to Option This is re-submission for rust-lang#59628. Renames `deref()` to `as_deref()` and adds `deref_mut()` impls and tests. CC rust-lang#50264 r? @Kimundi (I picked you as you're the previous reviewer.)
Introduce `as_deref` to Option This is re-submission for rust-lang#59628. Renames `deref()` to `as_deref()` and adds `deref_mut()` impls and tests. CC rust-lang#50264 r? @Kimundi (I picked you as you're the previous reviewer.)
Introduce `as_deref` to Option This is re-submission for rust-lang#59628. Renames `deref()` to `as_deref()` and adds `deref_mut()` impls and tests. CC rust-lang#50264 r? @Kimundi (I picked you as you're the previous reviewer.)
addresses #50264
renamed
deref()
->as_deref()
added
deref_mut()
impls + testsfixed breaking changes