Merge pull request #65 from evenfurther/clippy

style: fix clippy warnings
evenfurther · Dec 31, 2023 · f1dfb71 · f1dfb71
2 parents dc17d6b + ae66622
commit f1dfb71
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/aoc/src/input.rs b/aoc/src/input.rs
@@ -6,7 +6,7 @@ use std::str::FromStr;
 pub static mut OVERRIDE_INPUT: Option<String> = None;
 
 pub fn input_bytes(day: usize) -> Result<Vec<u8>, Error> {
-    match { unsafe { &OVERRIDE_INPUT } } {
+    match unsafe { &OVERRIDE_INPUT } {
         Some(s) => Ok(std::fs::read(s).unwrap_or_else(|_| {
             let mut s = s.as_bytes().to_vec();
             s.push(b'\n');
@@ -20,7 +20,7 @@ pub fn input_string(day: usize) -> Result<String, Error> {
     Ok(String::from_utf8(input_bytes(day)?)?)
 }
 
-/// Parse input as lines() if `sep` is absent, or as a single line
+/// Parse input as `lines()` if `sep` is absent, or as a single line
 /// if `sep` is present.
 pub fn parse_input<T>(input: &str, sep: Option<&str>) -> Result<Vec<T>, Error>
 where

diff --git a/aoc/src/run.rs b/aoc/src/run.rs
@@ -107,7 +107,7 @@ where
     if opts.input.is_some() && opts.all {
         eyre::bail!("--all and --input are not compatible");
     }
-    if opts.part.is_some_and(|p| p < 1 || p > 2) {
+    if opts.part.is_some_and(|p| !(1..=2).contains(&p)) {
         eyre::bail!("--part accepts argument must be 1 or 2");
     }
     unsafe {