Skip to content

Commit 4aa50c2

Browse files
committed
Address PR feedback
1 parent c2dcad5 commit 4aa50c2

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

sentry-android-replay/src/main/java/io/sentry/android/replay/DefaultReplayBreadcrumbConverter.kt

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,6 @@ public open class DefaultReplayBreadcrumbConverter : ReplayBreadcrumbConverter {
2323
}
2424

2525
override fun convert(breadcrumb: Breadcrumb): RRWebEvent? {
26-
var rrwebBreadcrumb: RRWebBreadcrumbEvent? = null
27-
2826
var breadcrumbMessage: String? = null
2927
var breadcrumbCategory: String? = null
3028
var breadcrumbLevel: SentryLevel? = null
@@ -97,8 +95,8 @@ public open class DefaultReplayBreadcrumbConverter : ReplayBreadcrumbConverter {
9795
breadcrumbData.putAll(breadcrumb.data)
9896
}
9997
}
100-
if (!breadcrumbCategory.isNullOrEmpty()) {
101-
rrwebBreadcrumb = RRWebBreadcrumbEvent().apply {
98+
return if (!breadcrumbCategory.isNullOrEmpty()) {
99+
RRWebBreadcrumbEvent().apply {
102100
timestamp = breadcrumb.timestamp.time
103101
breadcrumbTimestamp = breadcrumb.timestamp.time / 1000.0
104102
breadcrumbType = "default"
@@ -107,8 +105,9 @@ public open class DefaultReplayBreadcrumbConverter : ReplayBreadcrumbConverter {
107105
level = breadcrumbLevel
108106
data = breadcrumbData
109107
}
108+
} else {
109+
null
110110
}
111-
return rrwebBreadcrumb
112111
}
113112

114113
private fun Breadcrumb.isValidForRRWebSpan(): Boolean {

0 commit comments

Comments
 (0)