Fix clippy warnings

Author: Kobzol

collector/src/artifact_stats.rs

@@ -107,11 +107,11 @@ impl ArtifactStats {
 /// Normalizes the following things, in the following order:
 /// - Demangles the symbol.
 /// - Removes `.cold` and `.warm` from the end of the symbol, to merge cold and hot parts of a function
-/// into the same symbol.
+///   into the same symbol.
 /// - Removes rustc hashes from the symbol, e.g. `foo::[abcdef]` -> `foo::[]` or
-/// `foo::abcd` -> `foo`.
+///   `foo::abcd` -> `foo`.
 /// - Removes suffixes after a dot from the symbol, e.g. `anon.abcdef.123` -> `anon` or
-/// `foo.llvm.123` -> `foo`.
+///   `foo.llvm.123` -> `foo`.
 ///
 /// These modifications should remove things added by LLVM in the LTO/PGO phase.
 /// See more information here: https://rust-lang.github.io/rfcs/2603-rust-symbol-name-mangling-v0.html#vendor-specific-suffix

collector/src/bin/rustc-fake.rs

@@ -118,10 +118,7 @@ fn main() {
 
                 let prof_out_dir = create_self_profile_dir();
                 if wrapper == "PerfStatSelfProfile" {
-                    cmd.arg(&format!(
-                        "-Zself-profile={}",
-                        prof_out_dir.to_str().unwrap()
-                    ));
+                    cmd.arg(format!("-Zself-profile={}", prof_out_dir.to_str().unwrap()));
                     let _ = fs::remove_dir_all(&prof_out_dir);
                     let _ = fs::create_dir_all(&prof_out_dir);
                 }
@@ -189,10 +186,7 @@ fn main() {
 
                 let prof_out_dir = create_self_profile_dir();
                 if wrapper == "XperfStatSelfProfile" {
-                    tool.arg(&format!(
-                        "-Zself-profile={}",
-                        prof_out_dir.to_str().unwrap()
-                    ));
+                    tool.arg(format!("-Zself-profile={}", prof_out_dir.to_str().unwrap()));
                     let _ = fs::remove_dir_all(&prof_out_dir);
                     let _ = fs::create_dir_all(&prof_out_dir);
                 }

collector/src/compile/execute/bencher.rs

@@ -120,7 +120,7 @@ impl<'a> BenchProcessor<'a> {
     }
 }
 
-impl<'a> Processor for BenchProcessor<'a> {
+impl Processor for BenchProcessor<'_> {
     fn perf_tool(&self) -> PerfTool {
         if self.is_first_collection && self.is_self_profile {
             if cfg!(unix) {
@@ -309,7 +309,7 @@ impl SelfProfileS3Upload {
             .arg("INTELLIGENT_TIERING")
             .arg("--only-show-errors")
             .arg(upload.path())
-            .arg(&format!(
+            .arg(format!(
                 "s3://rustc-perf/{}",
                 &prefix.join(filename).to_str().unwrap()
             ))

collector/src/compile/execute/mod.rs

@@ -583,11 +583,7 @@ fn process_stat_output(
             // In any case it's better than crashing the collector and looping indefinitely trying
             // to to complete a run -- which happens if we propagate `parse_self_profile`'s errors
             // up to the caller.
-            if let Ok(self_profile_data) = parse_self_profile(dir, krate) {
-                self_profile_data
-            } else {
-                (None, None)
-            }
+            parse_self_profile(dir, krate).unwrap_or_default()
         }
         _ => (None, None),
     };
@@ -640,9 +636,11 @@ fn parse_self_profile(
     // `perf` pid. So just blindly look in the directory to hopefully find it.
     for entry in fs::read_dir(dir)? {
         let entry = entry?;
-        if entry.file_name().to_str().map_or(false, |s| {
-            s.starts_with(&crate_name) && s.ends_with("mm_profdata")
-        }) {
+        if entry
+            .file_name()
+            .to_str()
+            .is_some_and(|s| s.starts_with(&crate_name) && s.ends_with("mm_profdata"))
+        {
             full_path = Some(entry.path());
             break;
         }

collector/src/compile/execute/profiler.rs

@@ -106,7 +106,7 @@ impl<'a> ProfileProcessor<'a> {
     }
 }
 
-impl<'a> Processor for ProfileProcessor<'a> {
+impl Processor for ProfileProcessor<'_> {
     fn perf_tool(&self) -> PerfTool {
         PerfTool::ProfileTool(self.profiler)
     }

collector/src/compile/execute/rustc.rs

@@ -91,10 +91,10 @@ async fn record(
     .arg("--set")
     .arg("rust.deny-warnings=false")
     .arg("--set")
-    .arg(&format!("build.rustc={}", fake_rustc.to_str().unwrap()))
+    .arg(format!("build.rustc={}", fake_rustc.to_str().unwrap()))
     .env("RUSTC_PERF_REAL_RUSTC", &toolchain.components.rustc)
     .arg("--set")
-    .arg(&format!(
+    .arg(format!(
         "build.cargo={}",
         toolchain.components.cargo.to_str().unwrap()
     ))

collector/src/lib.rs

@@ -85,7 +85,7 @@ impl<'de> Deserialize<'de> for Bound {
     {
         struct BoundVisitor;
 
-        impl<'de> serde::de::Visitor<'de> for BoundVisitor {
+        impl serde::de::Visitor<'_> for BoundVisitor {
             type Value = Bound;
 
             fn visit_str<E>(self, value: &str) -> ::std::result::Result<Bound, E>

database/src/lib.rs

@@ -113,7 +113,7 @@ impl<'de> Deserialize<'de> for Date {
     {
         struct DateVisitor;
 
-        impl<'de> serde::de::Visitor<'de> for DateVisitor {
+        impl serde::de::Visitor<'_> for DateVisitor {
             type Value = Date;
 
             fn visit_str<E>(self, value: &str) -> ::std::result::Result<Date, E>

database/src/pool/postgres.rs

@@ -326,7 +326,7 @@ impl ConnectionManager for Postgres {
 }
 
 #[async_trait::async_trait]
-impl<'a> Transaction for PostgresTransaction<'a> {
+impl Transaction for PostgresTransaction<'_> {
     async fn commit(self: Box<Self>) -> Result<(), anyhow::Error> {
         Ok(self.conn.commit().await?)
     }

database/src/pool/sqlite.rs

@@ -20,7 +20,7 @@ pub struct SqliteTransaction<'a> {
 }
 
 #[async_trait::async_trait]
-impl<'a> Transaction for SqliteTransaction<'a> {
+impl Transaction for SqliteTransaction<'_> {
     async fn commit(mut self: Box<Self>) -> Result<(), anyhow::Error> {
         self.finished = true;
         Ok(self.conn.raw().execute_batch("COMMIT")?)