Add context to Error::EncodeError

src/enums.rs

@@ -82,7 +82,7 @@ pub enum Error {
     SystemTimeError(time::SystemTimeError),
     NotUtf8(Vec<u8>),
     DecodeError(Cow<'static, str>),
-    EncodeError(Cow<'static, str>),
+    EncodeError(String),
     PluginParsingError(PathBuf),
     PluginNotFound(String),
     TooManyActivePlugins {
@@ -148,7 +148,7 @@ impl fmt::Display for Error {
             Error::SystemTimeError(error) => error.fmt(f),
             Error::NotUtf8(bytes) => write!(f, "Expected a UTF-8 string, got bytes {bytes:02X?}"),
             Error::DecodeError(_) => write!(f, "Text could not be decoded from Windows-1252"),
-            Error::EncodeError(_) => write!(f, "Text could not be encoded in Windows-1252"),
+            Error::EncodeError(string) => write!(f, "The string \"{string}\" could not be encoded to Windows-1252"),
             Error::PluginParsingError(path) => {
                 write!(f, "An error was encountered while parsing the plugin at {path:?}")
             }

src/load_order/asterisk_based.rs

@@ -1021,7 +1021,7 @@ mod tests {
         load_order.plugins_mut().push(plugin);
 
         match load_order.save().unwrap_err() {
-            Error::EncodeError(s) => assert_eq!("unrepresentable character", s),
+            Error::EncodeError(s) => assert_eq!("Blȧnk.esm", s),
             e => panic!("Expected encode error, got {:?}", e),
         };
     }

src/load_order/mod.rs

@@ -42,9 +42,7 @@ fn strict_encode(string: &str) -> Result<Cow<'_, [u8]>, Error> {
     let (output, _, had_unmappable_chars) = WINDOWS_1252.encode(string);
 
     if had_unmappable_chars {
-        Err(Error::EncodeError(Cow::Borrowed(
-            "unrepresentable character",
-        )))
+        Err(Error::EncodeError(string.to_string()))
     } else {
         Ok(output)
     }

src/load_order/textfile_based.rs

@@ -806,7 +806,7 @@ mod tests {
         load_order.plugins_mut().push(plugin);
 
         match load_order.save().unwrap_err() {
-            Error::EncodeError(s) => assert_eq!("unrepresentable character", s),
+            Error::EncodeError(s) => assert_eq!("Blȧnk.esm", s),
             e => panic!("Expected encode error, got {:?}", e),
         };
     }

src/load_order/timestamp_based.rs

@@ -704,7 +704,7 @@ mod tests {
         load_order.plugins_mut().push(plugin);
 
         match load_order.save().unwrap_err() {
-            Error::EncodeError(s) => assert_eq!("unrepresentable character", s),
+            Error::EncodeError(s) => assert_eq!("Blȧnk.esm", s),
             e => panic!("Expected encode error, got {:?}", e),
         };
     }