Skip to content

Commit 58a342f

Browse files
committed
convert ApplicationExitInfoHistoryDispatcher.removeLatest() to use an iterator instead of for-each
1 parent 7f6dbc9 commit 58a342f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

sentry-android-core/src/main/java/io/sentry/android/core/ApplicationExitInfoHistoryDispatcher.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
import io.sentry.transport.ICurrentDateProvider;
1717
import java.util.ArrayList;
1818
import java.util.Collections;
19+
import java.util.Iterator;
1920
import java.util.List;
2021
import java.util.concurrent.TimeUnit;
2122
import org.jetbrains.annotations.ApiStatus;
@@ -131,9 +132,10 @@ private void waitPreviousSessionFlush() {
131132
@RequiresApi(api = Build.VERSION_CODES.R)
132133
private @Nullable ApplicationExitInfo removeLatest(
133134
final @NotNull List<ApplicationExitInfo> exitInfos) {
134-
for (ApplicationExitInfo applicationExitInfo : exitInfos) {
135+
for (Iterator<ApplicationExitInfo> it = exitInfos.iterator(); it.hasNext(); ) {
136+
ApplicationExitInfo applicationExitInfo = it.next();
135137
if (applicationExitInfo.getReason() == policy.getTargetReason()) {
136-
exitInfos.remove(applicationExitInfo);
138+
it.remove();
137139
return applicationExitInfo;
138140
}
139141
}

0 commit comments

Comments
 (0)