.

Author: taradepan

benches/bench_analyze.rs

@@ -83,7 +83,7 @@ fn bench_analyze(c: &mut Criterion) {
 
     c.bench_function("analyze_files_single_large_file", |b| {
         b.iter(|| {
-            let diags = analyze_files(black_box(&[big_path.clone()])).unwrap();
+            let diags = analyze_files(black_box(std::slice::from_ref(&big_path))).unwrap();
             black_box(diags);
         });
     });

src/fast_parser/lexer.rs

@@ -1063,8 +1063,8 @@ mod tests {
         let src = "if True:\n    x = 1\n";
         let toks = tokens(src);
         // Should contain: KwIf True : Newline Indent Name Eq Number Newline Dedent Eof
-        assert!(toks.iter().any(|t| *t == Token::Indent));
-        assert!(toks.iter().any(|t| *t == Token::Dedent));
+        assert!(toks.contains(&Token::Indent));
+        assert!(toks.contains(&Token::Dedent));
     }
 
     #[test]

src/fast_parser/parser.rs

@@ -1747,11 +1747,11 @@ mod tests {
     #[test]
     fn test_walrus_target_collected() {
         let s = stmts("def f():\n    x = (n := foo())\n");
-        if let StmtKind::FunctionDef(f) = &s[0].kind {
-            if let StmtKind::Assign { value, .. } = &f.body[0].kind {
-                let walrus: Vec<&str> = value.walrus.iter().map(|(n, _)| *n).collect();
-                assert!(walrus.contains(&"n"), "walrus target `n` not found");
-            }
+        if let StmtKind::FunctionDef(f) = &s[0].kind
+            && let StmtKind::Assign { value, .. } = &f.body[0].kind
+        {
+            let walrus: Vec<&str> = value.walrus.iter().map(|(n, _)| *n).collect();
+            assert!(walrus.contains(&"n"), "walrus target `n` not found");
         }
     }
 

src/names.rs

@@ -592,13 +592,7 @@ mod tests {
         out
     }
 
-    fn assigns_and_usages(src: &str) -> (HashMap<String, usize>, HashSet<String>) {
-        let stmts = parse(src);
-        let mut a = HashMap::new();
-        let mut u = HashSet::new();
-        collect_assigns_and_usages(&stmts, &mut a, &mut u);
-        (a, u)
-    }
+
 
     #[test]
     fn test_simple_name_usage() {