From 88bffefa34c85d4151a8a9844cf30f263c9744c6 Mon Sep 17 00:00:00 2001 From: Luca Palmieri Date: Mon, 29 Mar 2021 17:28:17 +0100 Subject: [PATCH] Remove #[must_use] from set_tracer_provider (#501) Set `set_tracer_provider` no longer returns a guard: it returns back the existing tracing provider. Therefore it should not be marked as `must_use`. --- opentelemetry/src/global/trace.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/opentelemetry/src/global/trace.rs b/opentelemetry/src/global/trace.rs index f095774aa1..1d09cde3a2 100644 --- a/opentelemetry/src/global/trace.rs +++ b/opentelemetry/src/global/trace.rs @@ -252,8 +252,10 @@ pub fn tracer_with_version(name: &'static str, version: &'static str) -> BoxedTr /// Sets the given [`TracerProvider`] instance as the current global provider. /// +/// It returns the [`TracerProvider`] instance that was previously mounted as global provider +/// (e.g. [`NoopTracerProvider`] if a provider had not been set before). +/// /// [`TracerProvider`]: crate::trace::TracerProvider -#[must_use] pub fn set_tracer_provider(new_provider: P) -> GlobalTracerProvider where S: trace::Span + Send + Sync,