diff options
author | Vincent Ambo <mail@tazj.in> | 2022-12-20T14·22+0300 |
---|---|---|
committer | clbot <clbot@tvl.fyi> | 2022-12-25T18·25+0000 |
commit | 71174f6626cbf100a8428ddc334681e4edfb45e6 (patch) | |
tree | 1e803137d755001700f0e900aa5173bbf43fdf3f /tvix/eval/benches/eval.rs | |
parent | 67d508f2ece710714ce8abf6f7deba1fd2440487 (diff) |
fix(tvix/eval): fix current clippy warnings r/5486
It's been a while since the last time, so quite a lot of stuff has accumulated here. Change-Id: I0762827c197b30a917ff470fd8ae8f220f6ba247 Reviewed-on: https://cl.tvl.fyi/c/depot/+/7597 Reviewed-by: grfn <grfn@gws.fyi> Autosubmit: tazjin <tazjin@tvl.su> Tested-by: BuildkiteCI
Diffstat (limited to 'tvix/eval/benches/eval.rs')
-rw-r--r-- | tvix/eval/benches/eval.rs | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/tvix/eval/benches/eval.rs b/tvix/eval/benches/eval.rs index 006038490df4..f85e3585f017 100644 --- a/tvix/eval/benches/eval.rs +++ b/tvix/eval/benches/eval.rs @@ -2,23 +2,23 @@ use criterion::{black_box, criterion_group, criterion_main, Criterion}; use itertools::Itertools; fn interpret(code: &str) { - tvix_eval::Evaluation::new(code).evaluate() + tvix_eval::Evaluation::new(code, None).evaluate(); } fn eval_literals(c: &mut Criterion) { c.bench_function("int", |b| { - b.iter(|| black_box(interpret("42", None, Default::default()))) + b.iter(|| { + interpret("42"); + black_box(()) + }) }); } fn eval_merge_attrs(c: &mut Criterion) { c.bench_function("merge small attrs", |b| { b.iter(|| { - black_box(interpret( - "{ a = 1; b = 2; } // { c = 3; }", - None, - Default::default(), - )) + interpret("{ a = 1; b = 2; } // { c = 3; }"); + black_box(()) }) }); @@ -28,7 +28,10 @@ fn eval_merge_attrs(c: &mut Criterion) { (0..10000).map(|n| format!("a{n} = {n};")).join(" ") ); let expr = format!("{large_attrs} // {{ c = 3; }}"); - b.iter(move || black_box(interpret(&expr, None, Default::default()))) + b.iter(move || { + interpret(&expr); + black_box(()) + }) }); } |