wip: check for error directly

Author: not-matthias

crates/criterion_compat/criterion_fork/src/analysis/mod.rs

@@ -297,12 +297,13 @@ mod codspeed {
     ) {
         let (uri, bench_name) = create_uri_and_name(id, c);
 
-        let result = ::codspeed::fifo::send_cmd(codspeed::fifo::Command::CurrentBenchmark {
+        if let Err(error) = ::codspeed::fifo::send_cmd(codspeed::fifo::Command::CurrentBenchmark {
             pid: std::process::id(),
             uri: uri.clone(),
-        });
-        if result.is_err() && std::env::var("CODSPEED_ENV").is_ok() {
-            eprintln!("Failed to send benchmark URI to runner: {error:?}");
+        }) {
+            if std::env::var("CODSPEED_ENV").is_ok() {
+                eprintln!("Failed to send benchmark URI to runner: {error:?}");
+            }
         }
 
         let avg_iter_per_round = iters.iter().sum::<f64>() / iters.len() as f64;

crates/divan_compat/divan_fork/src/divan.rs

@@ -428,12 +428,13 @@ mod codspeed {
             bench_context.samples.time_samples.iter().map(|s| s.duration.picos / 1_000).collect();
         let max_time_ns = bench_context.options.max_time.map(|t| t.as_nanos());
 
-        let result = ::codspeed::fifo::send_cmd(codspeed::fifo::Command::CurrentBenchmark {
+        if let Err(error) = ::codspeed::fifo::send_cmd(codspeed::fifo::Command::CurrentBenchmark {
             pid: std::process::id(),
             uri: uri.clone(),
-        });
-        if result.is_err() && std::env::var("CODSPEED_ENV").is_ok() {
-            eprintln!("Failed to send benchmark URI to runner: {error:?}");
+        }) {
+            if std::env::var("CODSPEED_ENV").is_ok() {
+                eprintln!("Failed to send benchmark URI to runner: {error:?}");
+            }
         }
 
         ::codspeed::walltime::collect_raw_walltime_results(