diff options
author | HampusM <hampus@hampusmat.com> | 2022-11-06 14:06:22 +0100 |
---|---|---|
committer | HampusM <hampus@hampusmat.com> | 2022-11-06 14:06:22 +0100 |
commit | 3c993aa73c93f0fe335ced78d9709b39cdbd1935 (patch) | |
tree | 5b8d1d3bf743fe2383687d172caaee08df69a08b /src/libs/intertrait/cast | |
parent | e665c2a3ca9b15f406c8e12e4a7ab372fcad4d36 (diff) |
refactor: improve cast error handling
Diffstat (limited to 'src/libs/intertrait/cast')
-rw-r--r-- | src/libs/intertrait/cast/arc.rs | 13 | ||||
-rw-r--r-- | src/libs/intertrait/cast/box.rs | 13 | ||||
-rw-r--r-- | src/libs/intertrait/cast/error.rs | 7 | ||||
-rw-r--r-- | src/libs/intertrait/cast/rc.rs | 13 |
4 files changed, 28 insertions, 18 deletions
diff --git a/src/libs/intertrait/cast/arc.rs b/src/libs/intertrait/cast/arc.rs index 1742c32..135cf64 100644 --- a/src/libs/intertrait/cast/arc.rs +++ b/src/libs/intertrait/cast/arc.rs @@ -31,16 +31,17 @@ impl<CastFromSelf: ?Sized + CastFromSync> CastArc for CastFromSelf self: Arc<Self>, ) -> Result<Arc<OtherTrait>, CastError> { - let caster = - get_caster::<OtherTrait>((*self).type_id()).ok_or(CastError::CastFailed { - from: type_name::<CastFromSelf>(), - to: type_name::<OtherTrait>(), - })?; + let caster = get_caster::<OtherTrait>((*self).type_id()) + .map_err(CastError::GetCasterFailed)?; let cast_arc = caster .opt_cast_arc .ok_or(CastError::NotArcCastable(type_name::<OtherTrait>()))?; - Ok(cast_arc(self.arc_any())) + cast_arc(self.arc_any()).map_err(|err| CastError::CastFailed { + source: err, + from: type_name::<Self>(), + to: type_name::<OtherTrait>(), + }) } } diff --git a/src/libs/intertrait/cast/box.rs b/src/libs/intertrait/cast/box.rs index 5694d97..67fd949 100644 --- a/src/libs/intertrait/cast/box.rs +++ b/src/libs/intertrait/cast/box.rs @@ -30,12 +30,13 @@ impl<CastFromSelf: ?Sized + CastFrom> CastBox for CastFromSelf self: Box<Self>, ) -> Result<Box<OtherTrait>, CastError> { - let caster = - get_caster::<OtherTrait>((*self).type_id()).ok_or(CastError::CastFailed { - from: type_name::<CastFromSelf>(), - to: type_name::<OtherTrait>(), - })?; + let caster = get_caster::<OtherTrait>((*self).type_id()) + .map_err(CastError::GetCasterFailed)?; - Ok((caster.cast_box)(self.box_any())) + (caster.cast_box)(self.box_any()).map_err(|err| CastError::CastFailed { + source: err, + from: type_name::<Self>(), + to: type_name::<OtherTrait>(), + }) } } diff --git a/src/libs/intertrait/cast/error.rs b/src/libs/intertrait/cast/error.rs index a834c05..e6d86a5 100644 --- a/src/libs/intertrait/cast/error.rs +++ b/src/libs/intertrait/cast/error.rs @@ -1,9 +1,16 @@ +use crate::libs::intertrait::{CasterError, GetCasterError}; + #[derive(thiserror::Error, Debug)] pub enum CastError { + #[error("Failed to get caster")] + GetCasterFailed(#[from] GetCasterError), + #[error("Failed to cast from trait {from} to trait {to}")] CastFailed { + #[source] + source: CasterError, from: &'static str, to: &'static str, }, diff --git a/src/libs/intertrait/cast/rc.rs b/src/libs/intertrait/cast/rc.rs index 805bcd7..ec70544 100644 --- a/src/libs/intertrait/cast/rc.rs +++ b/src/libs/intertrait/cast/rc.rs @@ -30,12 +30,13 @@ impl<CastFromSelf: ?Sized + CastFrom> CastRc for CastFromSelf self: Rc<Self>, ) -> Result<Rc<OtherTrait>, CastError> { - let caster = - get_caster::<OtherTrait>((*self).type_id()).ok_or(CastError::CastFailed { - from: type_name::<CastFromSelf>(), - to: type_name::<OtherTrait>(), - })?; + let caster = get_caster::<OtherTrait>((*self).type_id()) + .map_err(CastError::GetCasterFailed)?; - Ok((caster.cast_rc)(self.rc_any())) + (caster.cast_rc)(self.rc_any()).map_err(|err| CastError::CastFailed { + source: err, + from: type_name::<Self>(), + to: type_name::<OtherTrait>(), + }) } } |