about summary refs log tree commit diff
diff options
context:
space:
mode:
authormichal kostrubiec <fractalfirdev@gmail.com>2025-06-19 13:45:14 +0200
committermichal kostrubiec <fractalfirdev@gmail.com>2025-06-20 19:34:48 +0200
commitb2b117ee281c817e1edbe34429a5a77faa2f3ece (patch)
tree1b9dbf9953a824b2640eb8b11a55393e7a0de3a2
parented441b64ede18d483c65a6376e7bde8dea94294f (diff)
downloadrust-b2b117ee281c817e1edbe34429a5a77faa2f3ece.tar.gz
rust-b2b117ee281c817e1edbe34429a5a77faa2f3ece.zip
Fixed some clippy warnings.
-rw-r--r--build_system/src/abi_test.rs7
1 files changed, 5 insertions, 2 deletions
diff --git a/build_system/src/abi_test.rs b/build_system/src/abi_test.rs
index 421bc387ce8..3c1531be27a 100644
--- a/build_system/src/abi_test.rs
+++ b/build_system/src/abi_test.rs
@@ -13,13 +13,16 @@ fn show_usage() {
 
 pub fn run() -> Result<(), String> {
     let mut args = std::env::args().skip(2);
+    // FractalFir: In the future, I'd like to add some more subcommands / options.
+    // So, this loop ought to stay for that purpose. It should also stay as a while loop(to parse args)
+    #[allow(clippy::never_loop, clippy::while_let_on_iterator)]
     while let Some(arg) = args.next() {
         match arg.as_str() {
             "--help" => {
                 show_usage();
                 return Ok(());
             }
-            _ => return Err(format!("Unknown option {}", arg)),
+            _ => return Err(format!("Unknown option {arg:?}")),
         }
     }
     // Ensure that we have a cloned version of abi-cafe on hand.
@@ -56,7 +59,7 @@ pub fn run() -> Result<(), String> {
         &"c_calls_cg_gcc",
     ];
     // Run ABI cafe.
-    run_command_with_output(cmd, Some(&Path::new("clones/abi-cafe")))?;
+    run_command_with_output(cmd, Some(Path::new("clones/abi-cafe")))?;
 
     Ok(())
 }