Fixed warnings, changed MSRV to 1.81

Author: FractalFir

.github/workflows/ci.yml

@@ -26,7 +26,7 @@ jobs:
           rustup default $RUST_TOOLCHAIN_VERSION
           rustup component add --toolchain $RUST_TOOLCHAIN_VERSION rustfmt clippy
         env:
-          RUST_TOOLCHAIN_VERSION: 1.75.0
+          RUST_TOOLCHAIN_VERSION: 1.81.0
 
       - uses: Swatinem/rust-cache@v2
         with:

collector/src/compile/execute/mod.rs

@@ -152,31 +152,31 @@ fn performance_cores() -> Option<&'static String> {
 }
 #[cfg(target_os = "windows")]
 // Windows stub - TODO: implement to add Performance core support to Windows.
-fn performance_cores() -> Option<&'static String>{
-	None 
+fn performance_cores() -> Option<&'static String> {
+    None
 }
 #[cfg(target_os = "macos")]
 // To my knoweladge, MacOS does not run on hardware with P/E cores, so this can simply always return None.
-fn performance_cores() -> Option<&'static String>{
-	None 
+fn performance_cores() -> Option<&'static String> {
+    None
 }
-#[cfg(not(any(target_os = "windows",target_os = "macos",target_os = "linux")))]
+#[cfg(not(any(target_os = "windows", target_os = "macos", target_os = "linux")))]
 // I don't think rustc-perf runs on OSs other than Linux, Windows, or Mac, but I'll still leave a stub here just in case.
-fn performance_cores() -> Option<&'static String>{
-	None
+fn performance_cores() -> Option<&'static String> {
+    None
 }
 #[cfg(target_os = "linux")]
 /// Makes the benchmark run only on Performance cores.
-fn run_on_p_cores(path:&Path,cpu_list:&str)->Command{
-        	let mut cmd = Command::new("taskset");
-        	cmd.arg("--cpu-list");
-        	cmd.arg(cpu_list);
-        	cmd.arg(path);
-        	cmd
+fn run_on_p_cores(path: &Path, cpu_list: &str) -> Command {
+    let mut cmd = Command::new("taskset");
+    cmd.arg("--cpu-list");
+    cmd.arg(cpu_list);
+    cmd.arg(path);
+    cmd
 }
 #[cfg(not(target_os = "linux"))]
-fn run_on_p_cores(path:&Path,cpu_list:&str)->Command{
-	todo!("Can't run comamnds on the P cores on this platofmr")
+fn run_on_p_cores(_path: &Path, _cpu_list: &str) -> Command {
+    todo!("Can't run comamnds on the P cores on this platofmr")
 }
 impl<'a> CargoProcess<'a> {
     pub fn incremental(mut self, incremental: bool) -> Self {
@@ -198,9 +198,9 @@ impl<'a> CargoProcess<'a> {
     fn base_command(&self, cwd: &Path, subcommand: &str) -> Command {
         // Processors with P and E cores require special handling
         let mut cmd = if let Some(p_cores) = performance_cores() {
-        	run_on_p_cores(Path::new(&self.toolchain.components.cargo),p_cores)
-        } else{
-        	Command::new(Path::new(&self.toolchain.components.cargo))
+            run_on_p_cores(Path::new(&self.toolchain.components.cargo), p_cores)
+        } else {
+            Command::new(Path::new(&self.toolchain.components.cargo))
         };
         cmd
             // Not all cargo invocations (e.g. `cargo clean`) need all of these
@@ -606,8 +606,8 @@ fn process_stat_output(
         let _unit = get!(parts.next());
         let mut name = get!(parts.next());
         // Map P-core events to normal events
-        if name == "cpu_core/instructions:u/"{
-        	name = "instructions:u".into();
+        if name == "cpu_core/instructions:u/" {
+            name = "instructions:u".into();
         }
         let _time = get!(parts.next());
         let pct = get!(parts.next());