Tidy up TestRecorder/PrintRecorder

metrics-rs · Aug 25, 2024 · b85e835 · b85e835
1 parent 448b700
commit b85e835
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/metrics/benches/macros.rs b/metrics/benches/macros.rs
@@ -10,6 +10,7 @@ use rand::{thread_rng, Rng};
 
 #[derive(Default)]
 struct TestRecorder;
+
 impl Recorder for TestRecorder {
     fn describe_counter(&self, _: KeyName, _: Option<Unit>, _: SharedString) {}
     fn describe_gauge(&self, _: KeyName, _: Option<Unit>, _: SharedString) {}
@@ -59,27 +60,21 @@ fn macro_benchmark(c: &mut Criterion) {
         });
     });
     group.bench_function("local_initialized/no_labels", |b| {
-        let recorder = TestRecorder;
-
-        metrics::with_local_recorder(&recorder, || {
+        metrics::with_local_recorder(&TestRecorder, || {
             b.iter(|| {
                 counter!("counter_bench").increment(42);
             });
         });
     });
     group.bench_function("local_initialized/with_static_labels", |b| {
-        let recorder = TestRecorder;
-
-        metrics::with_local_recorder(&recorder, || {
+        metrics::with_local_recorder(&TestRecorder, || {
             b.iter(|| {
                 counter!("counter_bench", "request" => "http", "svc" => "admin").increment(42);
             });
         });
     });
     group.bench_function("local_initialized/with_dynamic_labels", |b| {
-        let recorder = TestRecorder;
-
-        metrics::with_local_recorder(&recorder, || {
+        metrics::with_local_recorder(&TestRecorder, || {
             let label_val = thread_rng().gen::<u64>().to_string();
             b.iter(move || {
                 counter!("counter_bench", "request" => "http", "uid" => label_val.clone())

diff --git a/metrics/examples/basic.rs b/metrics/examples/basic.rs
@@ -45,7 +45,6 @@ impl HistogramFn for PrintHandle {
     }
 }
 
-#[derive(Default)]
 struct PrintRecorder;
 
 impl Recorder for PrintRecorder {