subscriber: explain why we always call inner.register_callsite() be…

…fore if statement (#1433) This backports #1433 from `master`. * subscriber: explain why we always call `inner.register_callsite()` before if statement * Apply suggestions from code review Co-authored-by: Eliza Weisman <[email protected]>
tokio-rs · Jun 26, 2021 · ab62076 · ab62076
1 parent de76e55
commit ab62076
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/tracing-subscriber/src/layer.rs b/tracing-subscriber/src/layer.rs
@@ -736,6 +736,9 @@ where
             return outer;
         }
 
+        // The intention behind calling `inner.register_callsite()` before the if statement
+        // is to ensure that the inner subscriber is informed that the callsite exists
+        // regardless of the outer subscriber's filtering decision.
         let inner = self.inner.register_callsite(metadata);
         if outer.is_sometimes() {
             // if this interest is "sometimes", return "sometimes" to ensure that