feat: improved error type by requireing Sync + Send

Author: WolverinDEV

examples/dyn_memory/src/main.rs

@@ -11,7 +11,7 @@ use raw_struct::{
     Viewable,
 };
 
-fn main() -> Result<(), Box<dyn Error>> {
+fn main() -> Result<(), Box<dyn Error + Send + Sync>> {
     let buffer = [0x1122u64, 0x8877, 0x9988];
     let object = Copy::<dyn Container<u64>>::read_object(&buffer, 0x00)?;
 

examples/minimal/src/main.rs

@@ -17,7 +17,7 @@ use raw_struct::{
     Viewable,
 };
 
-fn main() -> Result<(), Box<dyn Error>> {
+fn main() -> Result<(), Box<dyn Error + Send + Sync>> {
     let mut memory = [0u8; 0x20];
     memory[0..4].copy_from_slice(&0x6Fu32.to_le_bytes());
     memory[4..8].copy_from_slice(&0x99u32.to_le_bytes());

raw_struct/src/error.rs

@@ -9,6 +9,8 @@ use core::fmt;
 #[cfg(not(feature = "no_std"))]
 pub use std::error::Error as ErrorType;
 
+pub type Error = Box<dyn ErrorType + Send + Sync + 'static>;
+
 #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord)]
 pub enum AccessMode {
     Read,
@@ -37,7 +39,7 @@ impl ErrorType for AccessViolation {}
 
 #[derive(Debug)]
 pub struct AccessError {
-    pub source: Box<dyn ErrorType + 'static>,
+    pub source: Error,
 
     pub offset: u64,
     pub size: usize,

raw_struct/src/memory.rs

@@ -8,20 +8,16 @@ use core::{
 };
 
 use crate::error::{
-    self,
     AccessViolation,
+    Error,
 };
 
 pub trait MemoryView {
-    fn read_memory(&self, offset: u64, buffer: &mut [u8]) -> Result<(), Box<dyn error::ErrorType>>;
+    fn read_memory(&self, offset: u64, buffer: &mut [u8]) -> Result<(), Error>;
 }
 
 impl<T: Copy> MemoryView for T {
-    fn read_memory(
-        &self,
-        offset: u64,
-        buffer: &mut [u8],
-    ) -> Result<(), alloc::boxed::Box<dyn error::ErrorType>> {
+    fn read_memory(&self, offset: u64, buffer: &mut [u8]) -> Result<(), Error> {
         let src_buffer = unsafe {
             core::slice::from_raw_parts(self as *const _ as *const u8, core::mem::size_of_val(self))
         };
@@ -37,12 +33,12 @@ impl<T: Copy> MemoryView for T {
 }
 
 pub trait FromMemoryView: Sized {
-    fn read_object(view: &dyn MemoryView, offset: u64) -> Result<Self, Box<dyn error::ErrorType>>;
+    fn read_object(view: &dyn MemoryView, offset: u64) -> Result<Self, Error>;
     // fn read_boxed(view: &dyn MemoryView, offset: u64) -> Result<Box<Self>, Box<dyn error::ErrorType>>;
 }
 
 impl<T: Copy> FromMemoryView for T {
-    fn read_object(view: &dyn MemoryView, offset: u64) -> Result<Self, Box<dyn error::ErrorType>> {
+    fn read_object(view: &dyn MemoryView, offset: u64) -> Result<Self, Error> {
         let mut result = MaybeUninit::uninit();
         let size = mem::size_of_val(&result);
 

raw_struct/src/reference.rs

@@ -1,7 +1,4 @@
-use alloc::{
-    boxed::Box,
-    sync::Arc,
-};
+use alloc::sync::Arc;
 use core::{
     self,
     marker,
@@ -10,8 +7,8 @@ use core::{
 
 use crate::{
     error::{
-        self,
         AccessError,
+        Error,
     },
     memory::MemoryView,
     view::ViewableImplementation,
@@ -37,7 +34,7 @@ impl ReferenceMemory {
 }
 
 impl MemoryView for ReferenceMemory {
-    fn read_memory(&self, offset: u64, buffer: &mut [u8]) -> Result<(), Box<dyn error::ErrorType>> {
+    fn read_memory(&self, offset: u64, buffer: &mut [u8]) -> Result<(), Error> {
         self.inner.read_memory(self.address + offset, buffer)
     }
 }