Skip to content

Commit 97d271d

Browse files
KM-15232: replace removeFirst with removeAt(0)
1 parent 941e643 commit 97d271d

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

kpi/src/commonMain/kotlin/com/privateinternetaccess/kpi/internals/KPI.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -372,8 +372,8 @@ internal class KPI(
372372

373373
// Update sample events. If we have more than the max of sample events supported.
374374
// Clear the oldest one before adding the new one.
375-
if (sampleEvents.size >= eventsHistorySize) {
376-
sampleEvents.removeFirst()
375+
if (sampleEvents.size >= eventsHistorySize && sampleEvents.isNotEmpty()) {
376+
sampleEvents.removeAt(0)
377377
}
378378
sampleEvents.add(event)
379379
}

kpi/src/commonMain/kotlin/com/privateinternetaccess/kpi/internals/KPIPersistencyImpl.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,8 @@ internal class KPIPersistencyImpl internal constructor(
9191

9292
// Update sample events.
9393
val recentEvents = sampleEvents().toMutableList()
94-
if (recentEvents.size > eventHistorySize) {
95-
recentEvents.removeFirst()
94+
if (recentEvents.size > eventHistorySize && recentEvents.isNotEmpty()) {
95+
recentEvents.removeAt(0)
9696
}
9797
recentEvents.add(event)
9898
val jsonString = json.encodeToString(

0 commit comments

Comments
 (0)