as_deref() and as_deref_mut() impls#59628
as_deref() and as_deref_mut() impls#59628U007D wants to merge 3 commits intorust-lang:masterfrom U007D:master
as_deref() and as_deref_mut() impls#59628Conversation
|
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