Add must_use attributes to the Uninstall structs

open-telemetry · Dec 1, 2020 · 941f6e7 · 941f6e7
1 parent 283e0b5
commit 941f6e7
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 0 deletions.
diff --git a/opentelemetry-contrib/src/trace/exporter/datadog/mod.rs b/opentelemetry-contrib/src/trace/exporter/datadog/mod.rs
@@ -290,5 +290,6 @@ impl trace::SpanExporter for DatadogExporter {
 }
 
 /// Uninstalls the Datadog pipeline on drop
+#[must_use]
 #[derive(Debug)]
 pub struct Uninstall(global::TracerProviderGuard);
diff --git a/opentelemetry-jaeger/src/lib.rs b/opentelemetry-jaeger/src/lib.rs
@@ -225,6 +225,7 @@ pub fn new_pipeline() -> PipelineBuilder {
 }
 
 /// Guard that uninstalls the Jaeger trace pipeline when dropped
+#[must_use]
 #[derive(Debug)]
 pub struct Uninstall(global::TracerProviderGuard);
 

diff --git a/opentelemetry-otlp/src/lib.rs b/opentelemetry-otlp/src/lib.rs
@@ -233,5 +233,6 @@ impl OtlpPipelineBuilder {
 }
 
 /// Uninstalls the OTLP pipeline on drop
+#[must_use]
 #[derive(Debug)]
 pub struct Uninstall(global::TracerProviderGuard);
diff --git a/opentelemetry-zipkin/src/lib.rs b/opentelemetry-zipkin/src/lib.rs
@@ -320,5 +320,6 @@ impl trace::SpanExporter for Exporter {
 }
 
 /// Uninstalls the Zipkin pipeline on drop.
+#[must_use]
 #[derive(Debug)]
 pub struct Uninstall(global::TracerProviderGuard);
diff --git a/opentelemetry/src/exporter/trace/stdout.rs b/opentelemetry/src/exporter/trace/stdout.rs
@@ -141,5 +141,6 @@ where
 }
 
 /// Uninstalls the stdout pipeline on drop.
+#[must_use]
 #[derive(Debug)]
 pub struct Uninstall(global::TracerProviderGuard);