fix: change if statement to let Some,

remove unnecessary import
unimarkup · Jan 14, 2022 · 5efc48e · 5efc48e
1 parent ade82b0
commit 5efc48e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/config.rs b/src/config.rs
@@ -296,11 +296,11 @@ pub enum HtmlMathmode {
 impl Config {
     /// [`validate_config`] validates if file and paths exist and if config does not contradict itself
     pub fn validate_config(&mut self) -> Result<(), UmError> {
-        if self.out_file.is_some() {
-            if !self.out_file.as_ref().unwrap().exists() {
+        if let Some(ref file) = self.out_file {
+            if !file.exists() {
                 let msg = format!(
                     "output_file: {} is not a valid file",
-                    self.out_file.as_ref().unwrap().to_str().unwrap()
+                    file.to_string_lossy()
                 );
                 return Err(UmError::Backend(BackendError::new(msg)));
             } else if !self.overwrite_out_files {

diff --git a/src/frontend/parser.rs b/src/frontend/parser.rs
@@ -4,7 +4,6 @@ use pest::{iterators::Pair, iterators::Pairs, Parser, Span};
 use pest_derive::Parser;
 use std::fs;
 
-use crate::config;
 use crate::um_error::UmError;
 use crate::{
     config::Config,