Address lints yet again lol

Author: gwennlbh

Cargo.lock

@@ -232,8 +232,7 @@ async fn receive_responses(
 
         if recurse
             && max_recursion_depth
-                .map(|limit| response.1.recursion_level < limit)
-                .unwrap_or(true)
+                .map_or(true, |limit| response.1.recursion_level < limit)
         {
             println!(
                 "recursing: {} has depth {} < {}",

lychee-bin/src/commands/check.rs

@@ -20,6 +20,8 @@ mod task_tests {
         ResponseBody {
             uri: Uri::try_from(uri).unwrap(),
             status,
+            recursion_level: 0,
+            subsequent_uris: vec![],
         }
     }
 

lychee-bin/src/formatters/response/task.rs

@@ -177,6 +177,7 @@ mod tests {
             cached: 0,
             success_map,
             excluded_map: HashMap::default(),
+            other_map: HashMap::default(),
             detailed_stats: false,
         };
 

lychee-bin/src/formatters/stats/compact.rs

@@ -145,6 +145,7 @@ mod tests {
             success_map: HashMap::default(),
             error_map,
             excluded_map: HashMap::default(),
+            other_map: HashMap::default(),
             detailed_stats: true,
         };
 

lychee-bin/src/formatters/stats/detailed.rs

@@ -223,7 +223,7 @@ mod tests {
             Status::Cached(CacheStatus::Error(Some(404))),
             InputSource::Stdin,
             vec![],
-             0,
+            0,
         );
         stats.add(response);
         stats

lychee-bin/src/formatters/stats/markdown.rs

@@ -65,7 +65,7 @@ mod tests {
     fn make_test_response(url_str: &str, source: InputSource) -> Response {
         let uri = Uri::from(make_test_url(url_str));
 
-        Response::new(uri, Status::Error(ErrorKind::InvalidUrlHost), source)
+        Response::new(uri, Status::Error(ErrorKind::InvalidUrlHost), source, vec![], 0)
     }
 
     #[test]

lychee-bin/src/formatters/stats/mod.rs

@@ -251,8 +251,6 @@ fn load_cache(cfg: &Config) -> Option<Cache> {
 fn run_main() -> Result<i32> {
     use std::process::exit;
 
-    console_subscriber::init();
-
     let opts = match load_config() {
         Ok(opts) => opts,
         Err(e) => {

lychee-bin/src/main.rs

@@ -90,12 +90,10 @@ impl ResponseStats {
 
     /// Check if a response was already checked and added to the stats
     pub(crate) fn was_checked(&self, response: &Response) -> bool {
-        vec![
-            self.success_map.get(response.source()),
+        [self.success_map.get(response.source()),
             self.error_map.get(response.source()),
             self.excluded_map.get(response.source()),
-            self.other_map.get(response.source()),
-        ]
+            self.other_map.get(response.source())]
         .iter()
         .any(|&x| x.is_some_and(|x| x.contains(response.body())))
     }

lychee-bin/src/stats.rs

@@ -477,7 +477,13 @@ impl Client {
         self.remap(uri)?;
 
         if self.is_excluded(uri) {
-            return Ok(Response::new(uri.clone(), Status::Excluded, source, vec![], recursion_level));
+            return Ok(Response::new(
+                uri.clone(),
+                Status::Excluded,
+                source,
+                vec![],
+                recursion_level,
+            ));
         }
 
         let mut subsequent_uris: Vec<Uri> = vec![];
@@ -494,7 +500,13 @@ impl Client {
             }
         };
 
-        Ok(Response::new(uri.clone(), status, source, subsequent_uris, recursion_level))
+        Ok(Response::new(
+            uri.clone(),
+            status,
+            source,
+            subsequent_uris,
+            recursion_level,
+        ))
     }
 
     /// Check a single file using the file checker.