Bump brunch / benchmark dev-dependency

canvas/Cargo.toml

@@ -16,7 +16,7 @@ image-texel = { path = "../texel", version = "0.3.0" }
 bytemuck = "1.1"
 
 [dev-dependencies]
-brunch = "0.2.5"
+brunch = "0.6.1"
 [dev-dependencies.image]
 version = "0.24"
 default-features = false

canvas/benchmarks/bitpack.rs

@@ -69,13 +69,10 @@ fn main() {
         },
     ];
 
-    let mut benches = tests.map(|convert| {
-        Bench::new("framebuf::conversion::main", &convert.name())
-            .with(convert.prepare().expect("Failed to setup benchmark"))
-    });
-
-    // Technically, we're not meant to call this directly but this makes me sad.. Why are we forced
-    // to use a macro to setup such a simple data structure. Not like the macro makes it possible
-    // to define any more complicated thing than a linear list as well..
-    brunch::analyze(&mut benches[..])
+    let mut benches = brunch::Benches::default();
+    benches.extend(tests.map(|convert| {
+        Bench::new(format!("framebuf::conversion::main::{}", convert.name()))
+            .run(convert.prepare().expect("Failed to setup benchmark"))
+    }));
+    benches.finish();
 }

canvas/benchmarks/conversion.rs

@@ -213,17 +213,14 @@ fn main() {
         },
     ];
 
-    let mut benches = tests.map(|convert| {
+    let mut benches = brunch::Benches::default();
+    benches.extend(tests.map(|convert| {
         let bench = match convert.prepare() {
             Ok(bench) => bench,
             Err(err) => panic!("Failed to setup benchmark {:?}: {:?}", convert, err),
         };
 
-        Bench::new("framebuf::conversion::main", &convert.name()).with(bench)
-    });
-
-    // Technically, we're not meant to call this directly but this makes me sad.. Why are we forced
-    // to use a macro to setup such a simple data structure. Not like the macro makes it possible
-    // to define any more complicated thing than a linear list as well..
-    brunch::analyze(&mut benches[..])
+        Bench::new(format!("framebuf::conversion::main::{}", convert.name())).run(bench)
+    }));
+    benches.finish();
 }

canvas/benchmarks/intcast.rs

@@ -101,13 +101,10 @@ fn main() {
         },
     ];
 
-    let mut benches = tests.map(|convert| {
-        Bench::new("framebuf::conversion::main", &convert.name())
-            .with(convert.prepare().expect("Failed to setup benchmark"))
-    });
-
-    // Technically, we're not meant to call this directly but this makes me sad.. Why are we forced
-    // to use a macro to setup such a simple data structure. Not like the macro makes it possible
-    // to define any more complicated thing than a linear list as well..
-    brunch::analyze(&mut benches[..])
+    let mut benches = brunch::Benches::default();
+    benches.extend(tests.map(|convert| {
+        Bench::new(format!("framebuf::conversion::main::{}", convert.name()))
+            .run(convert.prepare().expect("Failed to setup benchmark"))
+    }));
+    benches.finish();
 }