Don't try to compile benchmarks on unsupported platforms

WASM, iOS and Redox do not support `run_on_demand`, and compiling
criterion on WASM can be troublesome.

Author: madsmtm

Cargo.toml

@@ -132,9 +132,6 @@ redox_syscall = "0.7"
 
 [dev-dependencies]
 colorous = "1.0.12"
-criterion = { version = "0.8.1", default-features = false, features = [
-    "cargo_bench_support",
-] }
 web-time = "1.0.0"
 winit = "0.30.0"
 
@@ -151,6 +148,9 @@ features = ["jpeg"]
 [target.'cfg(not(target_family = "wasm"))'.dev-dependencies]
 # Turn rayon back on everywhere else; creating the separate entry resets the features to default.
 rayon = "1.5.1"
+criterion = { version = "0.8.1", default-features = false, features = [
+    "cargo_bench_support",
+] }
 
 [target.'cfg(target_family = "wasm")'.dev-dependencies]
 wasm-bindgen-test = "0.3"

benches/buffer_mut.rs

@@ -1,66 +1,80 @@
 #![allow(deprecated)] // TODO
 
-use criterion::{criterion_group, criterion_main, Criterion};
+#[cfg(not(any(
+    target_family = "wasm",
+    all(target_vendor = "apple", not(target_os = "macos")),
+    target_os = "redox"
+)))]
+fn buffer_mut(c: &mut criterion::Criterion) {
+    use criterion::black_box;
+    use softbuffer::{Context, Surface};
+    use std::num::NonZeroU32;
+    use winit::event_loop::ControlFlow;
+    use winit::platform::run_on_demand::EventLoopExtRunOnDemand;
 
-fn buffer_mut(c: &mut Criterion) {
-    #[cfg(target_family = "wasm")]
-    {
-        // Do nothing.
-        let _ = c;
-    }
-
-    #[cfg(not(target_family = "wasm"))]
-    {
-        use criterion::black_box;
-        use softbuffer::{Context, Surface};
-        use std::num::NonZeroU32;
-        use winit::event_loop::ControlFlow;
-        use winit::platform::run_on_demand::EventLoopExtRunOnDemand;
-
-        let mut evl = winit::event_loop::EventLoop::new().unwrap();
-        let context = Context::new(evl.owned_display_handle()).unwrap();
-        let window = evl
-            .create_window(winit::window::Window::default_attributes().with_visible(false))
-            .unwrap();
+    let mut evl = winit::event_loop::EventLoop::new().unwrap();
+    let context = Context::new(evl.owned_display_handle()).unwrap();
+    let window = evl
+        .create_window(winit::window::Window::default_attributes().with_visible(false))
+        .unwrap();
 
-        evl.run_on_demand(move |ev, elwt| {
-            elwt.set_control_flow(ControlFlow::Poll);
+    evl.run_on_demand(move |ev, elwt| {
+        elwt.set_control_flow(ControlFlow::Poll);
 
-            if let winit::event::Event::AboutToWait = ev {
-                elwt.exit();
+        if let winit::event::Event::AboutToWait = ev {
+            elwt.exit();
 
-                let mut surface = Surface::new(&context, &window).unwrap();
+            let mut surface = Surface::new(&context, &window).unwrap();
 
-                let size = window.inner_size();
-                surface
-                    .resize(
-                        NonZeroU32::new(size.width).unwrap(),
-                        NonZeroU32::new(size.height).unwrap(),
-                    )
-                    .unwrap();
+            let size = window.inner_size();
+            surface
+                .resize(
+                    NonZeroU32::new(size.width).unwrap(),
+                    NonZeroU32::new(size.height).unwrap(),
+                )
+                .unwrap();
 
-                c.bench_function("buffer_mut()", |b| {
-                    b.iter(|| {
-                        for _ in 0..500 {
-                            black_box(surface.buffer_mut().unwrap());
-                        }
-                    });
+            c.bench_function("buffer_mut()", |b| {
+                b.iter(|| {
+                    for _ in 0..500 {
+                        black_box(surface.buffer_mut().unwrap());
+                    }
                 });
+            });
 
-                c.bench_function("pixels_mut()", |b| {
-                    let mut buffer = surface.buffer_mut().unwrap();
-                    b.iter(|| {
-                        for _ in 0..500 {
-                            let x: &mut [u32] = &mut buffer;
-                            black_box(x);
-                        }
-                    });
+            c.bench_function("pixels_mut()", |b| {
+                let mut buffer = surface.buffer_mut().unwrap();
+                b.iter(|| {
+                    for _ in 0..500 {
+                        let x: &mut [u32] = &mut buffer;
+                        black_box(x);
+                    }
                 });
-            }
-        })
-        .unwrap();
-    }
+            });
+        }
+    })
+    .unwrap();
 }
 
-criterion_group!(benches, buffer_mut);
-criterion_main!(benches);
+#[cfg(not(any(
+    target_family = "wasm",
+    all(target_vendor = "apple", not(target_os = "macos")),
+    target_os = "redox"
+)))]
+criterion::criterion_group!(benches, buffer_mut);
+
+#[cfg(not(any(
+    target_family = "wasm",
+    all(target_vendor = "apple", not(target_os = "macos")),
+    target_os = "redox"
+)))]
+criterion::criterion_main!(benches);
+
+#[cfg(any(
+    target_family = "wasm",
+    all(target_vendor = "apple", not(target_os = "macos")),
+    target_os = "redox"
+))]
+fn main() {
+    panic!("unsupported on WASM, iOS and Redox");
+}