diff options
| author | Yuki Okushi <jtitor@2k36.org> | 2022-05-25 21:45:35 +0900 |
|---|---|---|
| committer | Yuki Okushi <jtitor@2k36.org> | 2022-06-02 07:24:14 +0900 |
| commit | 572c39000ba24a254637f8e53bb5933fbd5ebef7 (patch) | |
| tree | cf0dfc6e2d44cff7d0875fd237741f524076b34f /library/alloc/src | |
| parent | b5a2d27f8f59df6f2162e61461b41d6116d4815e (diff) | |
| download | rust-572c39000ba24a254637f8e53bb5933fbd5ebef7.tar.gz rust-572c39000ba24a254637f8e53bb5933fbd5ebef7.zip | |
Stabilize `box_into_pin`
Diffstat (limited to 'library/alloc/src')
| -rw-r--r-- | library/alloc/src/boxed.rs | 23 |
1 files changed, 21 insertions, 2 deletions
diff --git a/library/alloc/src/boxed.rs b/library/alloc/src/boxed.rs index 41f3b1fa3dd..e2c692b5299 100644 --- a/library/alloc/src/boxed.rs +++ b/library/alloc/src/boxed.rs @@ -1175,14 +1175,33 @@ impl<T: ?Sized, A: Allocator> Box<T, A> { /// This conversion does not allocate on the heap and happens in place. /// /// This is also available via [`From`]. - #[unstable(feature = "box_into_pin", issue = "62370")] + /// + /// # Notes + /// + /// It's not recommended that crates add an impl like `From<Box<T>> for Pin<T>`, + /// as it'll introduce an ambiguity when calling `Pin::from`. + /// A demonstration of such a poor impl is shown below. + /// + /// ```compile_fail + /// # use std::pin::Pin; + /// struct Foo; // A type defined in this crate. + /// impl From<Box<()>> for Pin<Foo> { + /// fn from(_: Box<()>) -> Pin<Foo> { + /// Pin::new(Foo) + /// } + /// } + /// + /// let foo = Box::new(()); + /// let bar = Pin::from(foo); + /// ``` + #[stable(feature = "box_into_pin", since = "1.63.0")] #[rustc_const_unstable(feature = "const_box", issue = "92521")] pub const fn into_pin(boxed: Self) -> Pin<Self> where A: 'static, { // It's not possible to move or replace the insides of a `Pin<Box<T>>` - // when `T: !Unpin`, so it's safe to pin it directly without any + // when `T: !Unpin`, so it's safe to pin it directly without any // additional requirements. unsafe { Pin::new_unchecked(boxed) } } |
