Skip to content

Commit bf07cbb

Browse files
impr: Slightly speed up adding breadcrumbs (#4984)
Slightly speed up adding breadcrumbs by only serializing the breadcrumb once before passing it to the scope observers.
1 parent aea42ca commit bf07cbb

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
# Changelog
22

3+
## Unreleased
4+
5+
### Improvements
6+
7+
- Slightly speed up adding breadcrumbs (#4984)
8+
39
## 8.47.0
410

511
> [!Important]

Sources/Sentry/SentryScope.m

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,8 +135,10 @@ - (void)addBreadcrumb:(SentryBreadcrumb *)crumb
135135

136136
_currentBreadcrumbIndex = (_currentBreadcrumbIndex + 1) % _maxBreadcrumbs;
137137

138+
// Serializing is expensive. Only do it once.
139+
NSDictionary<NSString *, id> *serializedBreadcrumb = [crumb serialize];
138140
for (id<SentryScopeObserver> observer in self.observers) {
139-
[observer addSerializedBreadcrumb:[crumb serialize]];
141+
[observer addSerializedBreadcrumb:serializedBreadcrumb];
140142
}
141143
}
142144
}

0 commit comments

Comments
 (0)