Fix clippy lints

notes.md

@@ -1,5 +1,8 @@
 # Things to do next
 - `check` complains about some of the defaults. Make sure default settings conform with opinions of `check`
+- refactor to move functionality like rofi to shell scripts
+  Instead provide more scriptable functionality
+- actual config instead of hardcoding values
 
 # Things to think about
 - Do I want to copy templates when creating a course?
@@ -7,13 +10,11 @@
   - sometimes I don't need them
   - annoying to do by hand
   - I don't know either way when creating the course
-- Move `course.toml` to `.course.toml`?
+
-  - git doesn't automatically track hidden files I think (both pro and con)
 
 # Things to do, if you are bored
 - Think about better parsing for Courses (semester)
 - Think about better error handling
-  - probably implement “bubbling up” of errors using the `?` operator
 - Think about better code separation/organization
 
 # Things learned

src/config.rs

@@ -9,14 +9,8 @@ pub struct Config {
     pub note_extensions: Vec<String>,
 }
 
-#[derive(StructOpt)]
+#[derive(StructOpt, Default)]
 pub struct ListOptions {
     #[structopt(short, long)]
     pub all: bool,
 }
-
-impl Default for ListOptions {
-    fn default() -> Self {
-        ListOptions { all: false }
-    }
-}

src/courses.rs

@@ -72,7 +72,7 @@ impl Course {
             let mut coursestring = String::new();
             handle = File::open(&path).unwrap();
             handle.read_to_string(&mut coursestring).unwrap();
-            let mut course: Course = toml::from_str(&coursestring).expect(&format!("{:?}", &path));
+            let mut course: Course = toml::from_str(&coursestring)?;
             course.path = path.parent().unwrap().to_path_buf();
             if options.all || course.is_current() {
                 courses.push(course);

src/main.rs

@@ -116,16 +116,12 @@ fn list_lectures(conf: &Config, options: ListOptions) -> Result<Vec<PathBuf>> {
 
     let lectures = lectures?
         .into_iter()
-        .filter_map(|path| {
+        .filter(|path| {
             if let Some(ext) = path.extension() {
                 let ext = ext.to_owned().to_string_lossy().into();
-                if conf.note_extensions.contains(&ext) {
+                conf.note_extensions.contains(&ext)
-                    Some(path)
             } else {
-                    None
+                false
-                }
-            } else {
-                None
             }
         })
         .collect();

src/rofi.rs

@@ -20,7 +20,7 @@ pub fn select_meeting(conf: &Config, options: &ListOptions) -> Result<()> {
     let meetings: Vec<_> = Course::list(conf, options)?
         .iter()
         .cloned()
-        .map(|c: Course| {
+        .flat_map(|c: Course| {
             if let Some(meetings) = c.meetings() {
                 meetings.clone()
             } else {
@@ -30,7 +30,6 @@ pub fn select_meeting(conf: &Config, options: &ListOptions) -> Result<()> {
             .map(move |m| (c.clone(), m.clone()))
             .collect::<Vec<_>>()
         })
-        .flatten()
         .collect();
     let meeting_names: Vec<_> = meetings
         .iter()

src/util.rs

@@ -4,7 +4,7 @@ use std::path::{Path, PathBuf};
 use std::time::SystemTime;
 
 pub fn to_snail_case(from: impl Into<String>) -> String {
-    from.into().replace(" ", "_").to_lowercase()
+    from.into().replace(' ', "_").to_lowercase()
 }
 
 //     pub fn to_title_case(from: &str) -> String {