Delegate from map_id to try_map_id

This commit is contained in:
Alan Egerton 2021-11-27 16:40:47 +00:00
parent 04f1c09f90
commit 9f714ef035
No known key found for this signature in database
GPG Key ID: 07CAC3CCA7E0643F
2 changed files with 7 additions and 57 deletions

View File

@ -5,9 +5,13 @@
pub trait IdFunctor: Sized {
type Inner;
fn map_id<F>(self, f: F) -> Self
#[inline]
fn map_id<F>(self, mut f: F) -> Self
where
F: FnMut(Self::Inner) -> Self::Inner;
F: FnMut(Self::Inner) -> Self::Inner,
{
self.try_map_id::<_, !>(|value| Ok(f(value))).into_ok()
}
fn try_map_id<F, E>(self, f: F) -> Result<Self, E>
where
@ -17,25 +21,6 @@ fn try_map_id<F, E>(self, f: F) -> Result<Self, E>
impl<T> IdFunctor for Box<T> {
type Inner = T;
#[inline]
fn map_id<F>(self, mut f: F) -> Self
where
F: FnMut(Self::Inner) -> Self::Inner,
{
let raw = Box::into_raw(self);
unsafe {
// SAFETY: The raw pointer points to a valid value of type `T`.
let value = ptr::read(raw);
// SAFETY: Converts `Box<T>` to `Box<MaybeUninit<T>>` which is the
// inverse of `Box::assume_init()` and should be safe.
let mut raw: Box<mem::MaybeUninit<T>> = Box::from_raw(raw.cast());
// SAFETY: Write the mapped value back into the `Box`.
raw.write(f(value));
// SAFETY: We just initialized `raw`.
raw.assume_init()
}
}
#[inline]
fn try_map_id<F, E>(self, mut f: F) -> Result<Self, E>
where
@ -59,26 +44,6 @@ fn try_map_id<F, E>(self, mut f: F) -> Result<Self, E>
impl<T> IdFunctor for Vec<T> {
type Inner = T;
#[inline]
fn map_id<F>(mut self, mut f: F) -> Self
where
F: FnMut(Self::Inner) -> Self::Inner,
{
// FIXME: We don't really care about panics here and leak
// far more than we should, but that should be fine for now.
let len = self.len();
unsafe {
self.set_len(0);
let start = self.as_mut_ptr();
for i in 0..len {
let p = start.add(i);
ptr::write(p, f(ptr::read(p)));
}
self.set_len(len);
}
self
}
#[inline]
fn try_map_id<F, E>(mut self, mut f: F) -> Result<Self, E>
where
@ -119,14 +84,6 @@ fn try_map_id<F, E>(mut self, mut f: F) -> Result<Self, E>
impl<T> IdFunctor for Box<[T]> {
type Inner = T;
#[inline]
fn map_id<F>(self, f: F) -> Self
where
F: FnMut(Self::Inner) -> Self::Inner,
{
Vec::from(self).map_id(f).into()
}
#[inline]
fn try_map_id<F, E>(self, f: F) -> Result<Self, E>
where
@ -139,14 +96,6 @@ fn try_map_id<F, E>(self, f: F) -> Result<Self, E>
impl<I: Idx, T> IdFunctor for IndexVec<I, T> {
type Inner = T;
#[inline]
fn map_id<F>(self, f: F) -> Self
where
F: FnMut(Self::Inner) -> Self::Inner,
{
IndexVec::from_raw(self.raw.map_id(f))
}
#[inline]
fn try_map_id<F, E>(self, f: F) -> Result<Self, E>
where

View File

@ -25,6 +25,7 @@
#![feature(once_cell)]
#![feature(test)]
#![feature(thread_id_value)]
#![feature(unwrap_infallible)]
#![allow(rustc::default_hash_types)]
#![deny(unaligned_references)]