Skip to content

Commit 27c5f9b

Browse files
committed
Merge #1471: Fix Clippy errors
53c8694 fix: clippy errors (Jose Celano) Pull request description: Fix Clippy errors. ACKs for top commit: josecelano: ACK 53c8694 Tree-SHA512: 3c3cf86e3c7f5762564eb5c3b8535029b3d1e31c538a8505446514e8939b5aaeb90f2c950f54907704dfdc9ab7c368f19096f9c678a59a78279f01accbf68022
2 parents 851a210 + 53c8694 commit 27c5f9b

4 files changed

Lines changed: 11 additions & 11 deletions

File tree

packages/http-tracker-core/src/statistics/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ pub fn describe_metrics() -> Metrics {
1919
metrics.metric_collection.describe_counter(
2020
&metric_name!(HTTP_TRACKER_CORE_REQUESTS_RECEIVED_TOTAL),
2121
Some(Unit::Count),
22-
Some(MetricDescription::new("Total number of HTTP requests received")),
22+
Some(&MetricDescription::new("Total number of HTTP requests received")),
2323
);
2424

2525
metrics

packages/metrics/src/metric_collection.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ impl MetricCollection {
5757

5858
// Counter-specific methods
5959

60-
pub fn describe_counter(&mut self, name: &MetricName, opt_unit: Option<Unit>, opt_description: Option<MetricDescription>) {
60+
pub fn describe_counter(&mut self, name: &MetricName, opt_unit: Option<Unit>, opt_description: Option<&MetricDescription>) {
6161
tracing::info!(target: METRICS_TARGET, type = "counter", name = name.to_string(), unit = ?opt_unit, description = ?opt_description);
6262
self.counters.ensure_metric_exists(name);
6363
}
@@ -99,7 +99,7 @@ impl MetricCollection {
9999

100100
// Gauge-specific methods
101101

102-
pub fn describe_gauge(&mut self, name: &MetricName, opt_unit: Option<Unit>, opt_description: Option<MetricDescription>) {
102+
pub fn describe_gauge(&mut self, name: &MetricName, opt_unit: Option<Unit>, opt_description: Option<&MetricDescription>) {
103103
tracing::info!(target: METRICS_TARGET, type = "gauge", name = name.to_string(), unit = ?opt_unit, description = ?opt_description);
104104
self.gauges.ensure_metric_exists(name);
105105
}

packages/udp-tracker-core/src/statistics/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ pub fn describe_metrics() -> Metrics {
1919
metrics.metric_collection.describe_counter(
2020
&metric_name!(UDP_TRACKER_CORE_REQUESTS_RECEIVED_TOTAL),
2121
Some(Unit::Count),
22-
Some(MetricDescription::new("Total number of UDP requests received")),
22+
Some(&MetricDescription::new("Total number of UDP requests received")),
2323
);
2424

2525
metrics

packages/udp-tracker-server/src/statistics/mod.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,43 +25,43 @@ pub fn describe_metrics() -> Metrics {
2525
metrics.metric_collection.describe_counter(
2626
&metric_name!(UDP_TRACKER_SERVER_REQUESTS_ABORTED_TOTAL),
2727
Some(Unit::Count),
28-
Some(MetricDescription::new("Total number of UDP requests aborted")),
28+
Some(&MetricDescription::new("Total number of UDP requests aborted")),
2929
);
3030

3131
metrics.metric_collection.describe_counter(
3232
&metric_name!(UDP_TRACKER_SERVER_REQUESTS_BANNED_TOTAL),
3333
Some(Unit::Count),
34-
Some(MetricDescription::new("Total number of UDP requests banned")),
34+
Some(&MetricDescription::new("Total number of UDP requests banned")),
3535
);
3636

3737
metrics.metric_collection.describe_counter(
3838
&metric_name!(UDP_TRACKER_SERVER_REQUESTS_RECEIVED_TOTAL),
3939
Some(Unit::Count),
40-
Some(MetricDescription::new("Total number of UDP requests received")),
40+
Some(&MetricDescription::new("Total number of UDP requests received")),
4141
);
4242

4343
metrics.metric_collection.describe_counter(
4444
&metric_name!(UDP_TRACKER_SERVER_REQUESTS_ACCEPTED_TOTAL),
4545
Some(Unit::Count),
46-
Some(MetricDescription::new("Total number of UDP requests accepted")),
46+
Some(&MetricDescription::new("Total number of UDP requests accepted")),
4747
);
4848

4949
metrics.metric_collection.describe_counter(
5050
&metric_name!(UDP_TRACKER_SERVER_RESPONSES_SENT_TOTAL),
5151
Some(Unit::Count),
52-
Some(MetricDescription::new("Total number of UDP responses sent")),
52+
Some(&MetricDescription::new("Total number of UDP responses sent")),
5353
);
5454

5555
metrics.metric_collection.describe_counter(
5656
&metric_name!(UDP_TRACKER_SERVER_ERRORS_TOTAL),
5757
Some(Unit::Count),
58-
Some(MetricDescription::new("Total number of errors processing UDP requests")),
58+
Some(&MetricDescription::new("Total number of errors processing UDP requests")),
5959
);
6060

6161
metrics.metric_collection.describe_gauge(
6262
&metric_name!(UDP_TRACKER_SERVER_PERFORMANCE_AVG_PROCESSING_TIME_NS),
6363
Some(Unit::Nanoseconds),
64-
Some(MetricDescription::new(
64+
Some(&MetricDescription::new(
6565
"Average time to process a UDP connect request in nanoseconds",
6666
)),
6767
);

0 commit comments

Comments
 (0)