Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upTracking issue for `IntoCow` stabilization #27735
Comments
aturon
added
T-libs
B-unstable
labels
Aug 12, 2015
sfackler
added
the
I-nominated
label
Dec 16, 2015
This comment has been minimized.
This comment has been minimized.
|
Seems like we can potentially deprecate this due to a lack of interest? |
This comment has been minimized.
This comment has been minimized.
|
This is being used in HashMap2 to provide an ergonomic CoW entry, but I think there's some silly issues that result. @apasel422 do you recall the details? |
This comment has been minimized.
This comment has been minimized.
|
Does |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
|
alexcrichton
added
final-comment-period
and removed
I-nominated
labels
Dec 17, 2015
This comment has been minimized.
This comment has been minimized.
|
The libs team discussed this in triage recently and the decision was to deprecate |
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Jan 15, 2016
alexcrichton
referenced this issue
Jan 15, 2016
Merged
std: Stabilize APIs for the 1.7 release #30943
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Jan 15, 2016
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Jan 15, 2016
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Jan 16, 2016
bors
added a commit
that referenced
this issue
Jan 16, 2016
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Jan 16, 2016
alexcrichton
added a commit
to alexcrichton/rust
that referenced
this issue
Jan 16, 2016
Manishearth
added a commit
to Manishearth/rust
that referenced
this issue
Jan 17, 2016
bors
closed this
in
#30943
Jan 17, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
aturon commentedAug 12, 2015
NOTE:
IntoCowhas been deprecated, but you can get the same conversion by using theCow::Ownedconstructor, like so:Cow::Owned(vec![1, 2, 3]).The
IntoCowtrait is a custom conversion trait for working withCow; the custom nature is needed to deal with trait coherence issues.Ultimately, the hope was that this trait could support ergonomic "ownership polymorphism" for things like the
entryAPI. That's not the case today due to coherence limitations, but may become possible with impl specialization or other changes to the trait system.The API needs a full re-evaluation before stabilization.