Merge pull request #139 from Jedsek/master

Better code and 0 warnings from clippy

src/gist.rs

@@ -51,7 +51,6 @@ pub struct Gist {
 
 #[derive(Deserialize, Debug)]
 pub struct GistFile {
-    filename: String,
     pub content: String,
     pub language: String,
 }

src/language.rs

@@ -152,7 +152,7 @@ impl CodeHighlight {
         theme_set
             .add_from_folder(&syntect_dir)
             .map_err(|_e| LostTheWay::ThemeError {
-                theme: String::from((&syntect_dir).to_str().unwrap()),
+                theme: String::from((syntect_dir).to_str().unwrap()),
             })
             .suggestion(format!(
                 "Make sure {:#?} is a valid directory that has .tmTheme files",
@@ -162,7 +162,7 @@ impl CodeHighlight {
         syntax_set
             .add_from_folder(&syntect_dir, true)
             .map_err(|_e| LostTheWay::ThemeError {
-                theme: String::from((&syntect_dir).to_str().unwrap()),
+                theme: String::from((syntect_dir).to_str().unwrap()),
             })
             .suggestion(format!(
                 "Make sure {:#?} is a valid directory that has .sublime-syntax files",

src/the_way/filter.rs

@@ -72,7 +72,7 @@ impl TheWay {
             }
             (None, None) => self.list_snippets_in_date_range(from_date, to_date),
         };
-        let snippets = match &filters.pattern {
+        match &filters.pattern {
             Some(pattern) => {
                 let regex = Regex::new(&pattern.to_string_lossy())?;
                 snippets.map(|snippets| {
@@ -87,7 +87,6 @@ impl TheWay {
                 })
             }
             None => snippets,
-        };
-        snippets
+        }
     }
 }

src/the_way/search.rs

@@ -45,7 +45,7 @@ impl SkimItem for SearchCode {
     }
 }
 
-impl<'a> SkimItem for SearchSnippet {
+impl SkimItem for SearchSnippet {
     fn text(&self) -> Cow<str> {
         AnsiString::parse(&self.text_highlight).into_inner()
             + AnsiString::parse(&self.code.code_highlight).into_inner()

src/utils.rs

@@ -44,7 +44,6 @@ pub fn copy_to_clipboard(copy_cmd_field: &Option<String>, text: &str) -> color_e
     let copy_cmd_vec = copy_cmd_field
         .as_ref()
         .ok_or(LostTheWay::NoDefaultCopyCommand)?
-        .trim()
         .split_whitespace()
         .map(|s| s.to_owned())
         .collect::<Vec<String>>();