Don't loop through the flushed spans array needlessly

embrace-io · Mar 12, 2024 · f2bbde3 · f2bbde3
1 parent 303c83b
commit f2bbde3
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/...ce-android-sdk/src/main/java/io/embrace/android/embracesdk/internal/spans/SpanSinkImpl.kt b/...ce-android-sdk/src/main/java/io/embrace/android/embracesdk/internal/spans/SpanSinkImpl.kt
@@ -28,9 +28,7 @@ internal class SpanSinkImpl : SpanSink {
  override fun flushSpans(): List<EmbraceSpanData> {
  synchronized(spansToFlush) {
  spansToFlush.set(completedSpans())
- repeat(spansToFlush.get().size) {
- completedSpans.removeAll(spansToFlush.get().toSet())
- }
+ completedSpans.removeAll(spansToFlush.get().toSet())
  return spansToFlush.get()
  }
  }