Merge pull request #91 from epage/black

fix: Don't print when benchmarking
rosetta-rs · Feb 7, 2024 · b1d0c92 · b1d0c92
2 parents 985e4e2 + 101ff88
commit b1d0c92
Show file tree

Hide file tree

Showing 12 changed files with 129 additions and 81 deletions.
diff --git a/examples/argh-app/app.rs b/examples/argh-app/app.rs
@@ -31,12 +31,16 @@ fn parse_width(s: &str) -> Result<u32, String> {
 
 fn main() {
     let args: AppArgs = argh::from_env();
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/bpaf-app/app.rs b/examples/bpaf-app/app.rs
@@ -45,12 +45,16 @@ fn main() {
 
     let args = parser.run();
 
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/bpaf_derive-app/app.rs b/examples/bpaf_derive-app/app.rs
@@ -31,12 +31,16 @@ fn valid_width(width: &u32) -> bool {
 fn main() {
     let args = app_args().run();
 
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/clap-app/app.rs b/examples/clap-app/app.rs
@@ -57,12 +57,16 @@ fn main() {
             .collect(),
     };
 
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/clap-minimal-app/app.rs b/examples/clap-minimal-app/app.rs
@@ -57,12 +57,16 @@ fn main() {
             .collect(),
     };
 
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/clap_derive-app/app.rs b/examples/clap_derive-app/app.rs
@@ -28,12 +28,16 @@ fn parse_width(s: &str) -> Result<u32, String> {
 
 fn main() {
     let args = AppArgs::parse();
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/clap_lex-app/app.rs b/examples/clap_lex-app/app.rs
@@ -37,14 +37,18 @@ fn main() {
             std::process::exit(1);
         }
     };
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
 
 fn parse_args() -> Result<AppArgs, BoxedError> {

diff --git a/examples/gumdrop-app/app.rs b/examples/gumdrop-app/app.rs
@@ -34,12 +34,16 @@ fn parse_width(s: &str) -> Result<u32, String> {
 
 fn main() {
     let args = AppArgs::parse_args_default_or_exit();
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/lexopt-app/app.rs b/examples/lexopt-app/app.rs
@@ -35,14 +35,18 @@ fn main() {
             std::process::exit(1);
         }
     };
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
 
 fn parse_args() -> Result<AppArgs, lexopt::Error> {

diff --git a/examples/null-app/app.rs b/examples/null-app/app.rs
@@ -1,8 +1,12 @@
 fn main() {
     let args: Vec<_> = std::env::args_os().collect();
-    if 10 < args.len() {
-        println!("{:#?}", args.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        if 10 < args.len() {
+            println!("{:#?}", args.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
diff --git a/examples/pico-args-app/app.rs b/examples/pico-args-app/app.rs
@@ -46,14 +46,18 @@ fn main() {
         }
     };
 
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }
 
 fn parse_args() -> Result<AppArgs, pico_args::Error> {

diff --git a/examples/xflags-app/app.rs b/examples/xflags-app/app.rs
@@ -72,12 +72,16 @@ fn main() {
             err.exit();
         }
     }
-    println!("{:#?}", args.number);
-    println!("{:#?}", args.opt_number);
-    println!("{:#?}", args.width);
-    if 10 < args.input.len() {
-        println!("{:#?}", args.input.len());
-    } else {
-        println!("{:#?}", args);
+    #[cfg(debug_assertions)]
+    {
+        println!("{:#?}", args.number);
+        println!("{:#?}", args.opt_number);
+        println!("{:#?}", args.width);
+        if 10 < args.input.len() {
+            println!("{:#?}", args.input.len());
+        } else {
+            println!("{:#?}", args);
+        }
     }
+    std::hint::black_box(args);
 }