ref: Add comment why no lock required in scope (#4648)

Add a short comment to explain why we don't need to call synchronize spanLock when building the trace context in SentryScope.applyToEvent.
getsentry · Dec 19, 2024 · 6da2718 · 6da2718
1 parent e304199
commit 6da2718
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/Sources/Sentry/SentryScope.m b/Sources/Sentry/SentryScope.m
@@ -606,6 +606,8 @@ - (SentryEvent *__nullable)applyToEvent:(SentryEvent *)event
         }
     }
 
+    // We don't need call synchronized(_spanLock) here because we get a copy of the span in the
+    // _spanLock above.
     newContext[@"trace"] = [self buildTraceContext:span];
 
     event.context = newContext;