Improve `lightmotif-io` error type with `Arc` for internal errors and support for `Clone`

lightmotif-io/src/error.rs

 use std::fmt::Display;
 use std::fmt::Formatter;
+use std::sync::Arc;
 
 use nom::error::Error as NomError;
 
-#[derive(Debug)]
+#[derive(Clone, Debug)]
 pub enum Error {
     InvalidData,
-    Io(std::io::Error),
-    Nom(NomError<String>),
+    Io(Arc<std::io::Error>),
+    Nom(Arc<NomError<String>>),
 }
 
 impl From<lightmotif::err::InvalidData> for Error {
@@ -18,13 +19,13 @@ impl From<lightmotif::err::InvalidData> for Error {
 
 impl From<std::io::Error> for Error {
     fn from(error: std::io::Error) -> Self {
-        Error::Io(error)
+        Error::Io(Arc::new(error))
     }
 }
 
 impl From<NomError<&'_ str>> for Error {
     fn from(error: NomError<&'_ str>) -> Self {
-        Error::Nom(NomError::new(error.input.to_string(), error.code))
+        Error::Nom(Arc::new(NomError::new(error.input.to_string(), error.code)))
     }
 }
 

lightmotif-io/src/jaspar/mod.rs

@@ -96,7 +96,7 @@ impl<B: BufRead> Iterator for Reader<B> {
                 let text = match std::str::from_utf8(bytes) {
                     Ok(text) => text,
                     Err(_) => {
-                        return Some(Err(Error::Io(std::io::Error::new(
+                        return Some(Err(Error::from(std::io::Error::new(
                             std::io::ErrorKind::InvalidData,
                             "decoding error",
                         ))));

lightmotif-io/src/jaspar16/mod.rs

@@ -94,7 +94,7 @@ impl<B: BufRead, A: Alphabet> Iterator for Reader<B, A> {
                 let text = match std::str::from_utf8(bytes) {
                     Ok(text) => text,
                     Err(_) => {
-                        return Some(Err(Error::Io(std::io::Error::new(
+                        return Some(Err(Error::from(std::io::Error::new(
                             std::io::ErrorKind::InvalidData,
                             "decoding error",
                         ))));