Missing ReadExact & WriteAll error impls for std

- Moves `to_std_error` to embedded-io behind the std feature
- Add `ReadExactError` for `std::io::Error`
- Add `WriteAllError` for `std::io::Error`

Author: MabezDev

embedded-io-adapters/src/std.rs

@@ -1,5 +1,7 @@
 //! Adapters to/from `std::io` traits.
 
+use embedded_io::to_std_error;
+
 /// Adapter from `std::io` traits.
 #[derive(Clone)]
 pub struct FromStd<T: ?Sized> {
@@ -114,26 +116,3 @@ impl<T: embedded_io::Seek + ?Sized> std::io::Seek for ToStd<T> {
         self.inner.seek(pos.into()).map_err(to_std_error)
     }
 }
-
-fn to_std_error<T: embedded_io::Error>(err: T) -> std::io::Error {
-    let kind = match err.kind() {
-        embedded_io::ErrorKind::NotFound => std::io::ErrorKind::NotFound,
-        embedded_io::ErrorKind::PermissionDenied => std::io::ErrorKind::PermissionDenied,
-        embedded_io::ErrorKind::ConnectionRefused => std::io::ErrorKind::ConnectionRefused,
-        embedded_io::ErrorKind::ConnectionReset => std::io::ErrorKind::ConnectionReset,
-        embedded_io::ErrorKind::ConnectionAborted => std::io::ErrorKind::ConnectionAborted,
-        embedded_io::ErrorKind::NotConnected => std::io::ErrorKind::NotConnected,
-        embedded_io::ErrorKind::AddrInUse => std::io::ErrorKind::AddrInUse,
-        embedded_io::ErrorKind::AddrNotAvailable => std::io::ErrorKind::AddrNotAvailable,
-        embedded_io::ErrorKind::BrokenPipe => std::io::ErrorKind::BrokenPipe,
-        embedded_io::ErrorKind::AlreadyExists => std::io::ErrorKind::AlreadyExists,
-        embedded_io::ErrorKind::InvalidInput => std::io::ErrorKind::InvalidInput,
-        embedded_io::ErrorKind::InvalidData => std::io::ErrorKind::InvalidData,
-        embedded_io::ErrorKind::TimedOut => std::io::ErrorKind::TimedOut,
-        embedded_io::ErrorKind::Interrupted => std::io::ErrorKind::Interrupted,
-        embedded_io::ErrorKind::Unsupported => std::io::ErrorKind::Unsupported,
-        embedded_io::ErrorKind::OutOfMemory => std::io::ErrorKind::OutOfMemory,
-        _ => std::io::ErrorKind::Other,
-    };
-    std::io::Error::new(kind, format!("{:?}", err))
-}

embedded-io/CHANGELOG.md

@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Moved `embedded_io::blocking` to the crate root.
 - Split async traits to the `embedded-io-async` crate.
 - Split trait adapters to the `embedded-io-adapters` crate.
+- Move `to_std_error` to embedded-io and add `std::io` impls for `ReadExactError` & `WriteAllError`.
 - Rename trait `Io` to `ErrorKind`, for consistency with `embedded-hal`.
 
 ## 0.4.0 - 2022-11-25

embedded-io/src/lib.rs

@@ -227,12 +227,61 @@ impl<E> From<E> for ReadExactError<E> {
     }
 }
 
+#[cfg(feature = "std")]
+impl From<ReadExactError<std::io::Error>> for std::io::Error {
+    fn from(err: ReadExactError<std::io::Error>) -> Self {
+        match err {
+            ReadExactError::UnexpectedEof => {
+                std::io::Error::new(std::io::ErrorKind::UnexpectedEof, "UnexpectedEof".to_owned())
+            }
+            ReadExactError::Other(e) => to_std_error(e),
+        }
+    }
+}
+
+#[cfg(feature = "std")]
+impl From<WriteAllError<std::io::Error>> for std::io::Error {
+    fn from(err: WriteAllError<std::io::Error>) -> Self {
+        match err {
+            WriteAllError::WriteZero => {
+                std::io::Error::new(std::io::ErrorKind::WriteZero, "WriteZero".to_owned())
+            }
+            WriteAllError::Other(e) => to_std_error(e),
+        }
+    }
+}
+
 impl<E: fmt::Debug> fmt::Display for ReadExactError<E> {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         write!(f, "{:?}", self)
     }
 }
 
+#[cfg(feature = "std")]
+/// Convert a embedded-io error to a std::io::Error
+pub fn to_std_error<T: crate::Error>(err: T) -> std::io::Error {
+    let kind = match err.kind() {
+        crate::ErrorKind::NotFound => std::io::ErrorKind::NotFound,
+        crate::ErrorKind::PermissionDenied => std::io::ErrorKind::PermissionDenied,
+        crate::ErrorKind::ConnectionRefused => std::io::ErrorKind::ConnectionRefused,
+        crate::ErrorKind::ConnectionReset => std::io::ErrorKind::ConnectionReset,
+        crate::ErrorKind::ConnectionAborted => std::io::ErrorKind::ConnectionAborted,
+        crate::ErrorKind::NotConnected => std::io::ErrorKind::NotConnected,
+        crate::ErrorKind::AddrInUse => std::io::ErrorKind::AddrInUse,
+        crate::ErrorKind::AddrNotAvailable => std::io::ErrorKind::AddrNotAvailable,
+        crate::ErrorKind::BrokenPipe => std::io::ErrorKind::BrokenPipe,
+        crate::ErrorKind::AlreadyExists => std::io::ErrorKind::AlreadyExists,
+        crate::ErrorKind::InvalidInput => std::io::ErrorKind::InvalidInput,
+        crate::ErrorKind::InvalidData => std::io::ErrorKind::InvalidData,
+        crate::ErrorKind::TimedOut => std::io::ErrorKind::TimedOut,
+        crate::ErrorKind::Interrupted => std::io::ErrorKind::Interrupted,
+        crate::ErrorKind::Unsupported => std::io::ErrorKind::Unsupported,
+        crate::ErrorKind::OutOfMemory => std::io::ErrorKind::OutOfMemory,
+        _ => std::io::ErrorKind::Other,
+    };
+    std::io::Error::new(kind, format!("{:?}", err))
+}
+
 #[cfg(feature = "std")]
 impl<E: fmt::Debug> std::error::Error for ReadExactError<E> {}