Implement const Deref{,Mut} for String#155640
Implement const Deref{,Mut} for String#155640tshakalekholoane wants to merge 1 commit intorust-lang:mainfrom
Conversation
|
r? @jhpratt rustbot has assigned @jhpratt. Use Why was this reviewer chosen?The reviewer was selected based on:
|
273a00c to
ed1e03f
Compare
This comment has been minimized.
This comment has been minimized.
ed1e03f to
40b79f4
Compare
|
Apologies if there was previously a ruling on this, but waaaayyy back in the day we considered an allocation-free coercion from Providing |
|
cc @rust-lang/libs ^^ |
|
Oh, I was not aware of that. Thank you for the context, @cramertj. But Is there a distinction between the two that I am missing? |
|
I don't think |
|
FWIW I agree with the points in #155816 that this PR goes too far -- only under truly exceptional circumstances should we do anything with const traits in |
Implements
const Deref{,Mut}forString.Tracking issue: #143773.