Merge #1358

1358: apply profile filter to top-level entries only r=matklad a=matklad



Co-authored-by: Aleksey Kladov <[email protected]>

1 files changed, 6 insertions, 7 deletions

diff --git a/crates/ra_prof/src/lib.rs b/crates/ra_prof/src/lib.rs
index 031405879..e681b8e0b 100644
--- a/crates/ra_prof/src/lib.rs
+++ b/crates/ra_prof/src/lib.rs
@@ -76,17 +76,16 @@ pub fn profile(desc: &str) -> Profiler {
                 }
             };
         }
-
         if stack.starts.len() > stack.filter_data.depth {
             return Profiler { desc: None };
         }
-
-        if stack.filter_data.allowed.is_empty() || stack.filter_data.allowed.contains(desc) {
-            stack.starts.push(Instant::now());
-            Profiler { desc: Some(desc.to_string()) }
-        } else {
-            Profiler { desc: None }
+        let allowed = &stack.filter_data.allowed;
+        if stack.starts.is_empty() && !allowed.is_empty() && !allowed.contains(desc) {
+            return Profiler { desc: None };
         }
+
+        stack.starts.push(Instant::now());
+        Profiler { desc: Some(desc.to_string()) }
     })
 }