Rename error variants

src/delimiter.rs

@@ -27,14 +27,12 @@ impl Delimiter {
                 return Ok(Delimiter::Character(b'\t'));
             }
             let mut chars = s.chars();
-            let c = chars
-                .next()
-                .ok_or_else(|| CsvlensError::DelimiterEmptyError)?;
+            let c = chars.next().ok_or_else(|| CsvlensError::DelimiterEmpty)?;
             if !c.is_ascii() {
-                return Err(CsvlensError::DelimiterNotAsciiError(c));
+                return Err(CsvlensError::DelimiterNotAscii(c));
             }
             if chars.next().is_some() {
-                return Err(CsvlensError::DelimiterMultipleCharactersError(s.clone()));
+                return Err(CsvlensError::DelimiterMultipleCharacters(s.clone()));
             }
             Ok(Delimiter::Character(c.try_into()?))
         } else {

src/errors.rs

@@ -5,29 +5,29 @@ pub type CsvlensResult<T> = std::result::Result<T, CsvlensError>;
 /// Errors csvlens can have
 #[derive(Debug, Error)]
 pub enum CsvlensError {
-    #[error("Failed to read file: {0}")]
-    FileReadError(String),
+    #[error("File not found: {0}")]
+    FileNotFound(String),
 
     #[error("Column name not found: {0}")]
     ColumnNameNotFound(String),
 
     #[error("Delimiter should not be empty")]
-    DelimiterEmptyError,
+    DelimiterEmpty,
 
     #[error("Delimiter should be within the ASCII range: {0} is too fancy")]
-    DelimiterNotAsciiError(char),
+    DelimiterNotAscii(char),
 
     #[error("Delimiter should be exactly one character (or \\t), got '{0}'")]
-    DelimiterMultipleCharactersError(String),
+    DelimiterMultipleCharacters(String),
 
     #[error(transparent)]
-    DelimiterParseError(#[from] std::char::TryFromCharError),
+    DelimiterParsing(#[from] std::char::TryFromCharError),
 
     #[error(transparent)]
-    CsvError(#[from] csv::Error),
+    Csv(#[from] csv::Error),
 
     #[error(transparent)]
-    ArrowError(#[from] arrow::error::ArrowError),
+    Arrow(#[from] arrow::error::ArrowError),
 
     #[error(transparent)]
     Io(#[from] std::io::Error),

src/io.rs

@@ -15,8 +15,10 @@ impl SeekableFile {
         let inner_file_res;
 
         if let Some(filename) = maybe_filename {
-            let mut f =
-                File::open(filename).map_err(|_| CsvlensError::FileReadError(filename.clone()))?;
+            let mut f = File::open(filename).map_err(|e| match e.kind() {
+                std::io::ErrorKind::NotFound => CsvlensError::FileNotFound(filename.clone()),
+                _ => e.into(),
+            })?;
             // If not seekable, it most likely is due to process substitution using
             // pipe - write out to a temp file to make it seekable
             if f.seek(SeekFrom::Start(0)).is_err() {