On Wed Jul 2, 2025 at 5:27 PM CEST, Alice Ryhl wrote: > On Wed, Jul 2, 2025 at 5:07 PM Benno Lossin <[email protected]> wrote: >> On Wed Jul 2, 2025 at 3:18 PM CEST, Andreas Hindborg wrote: >> > +impl<T: Copy> OnceLock<T> { >> > + /// Get a copy of the contained object. >> > + /// >> > + /// Returns [`None`] if the [`OnceLock`] is empty. >> > + pub fn copy(&self) -> Option<T> { >> > + if self.init.load(Acquire) == 2 { >> > + // SAFETY: As determined by the load above, the object is >> > ready for shared access. >> > + Some(unsafe { *self.value.get() }) >> > + } else { >> > + None >> > + } >> >> The impl can just be: >> >> self.as_ref().copied() >> >> Would it make sense for this function to take `self` instead & we make >> the `OnceLock` also `Copy` if `T: Copy`? Maybe not... > > Atomics are not Copy.
Ah right... Yeah it probably also isn't useful. --- Cheers, Benno
