Do not use sublist as it crashes if index > items.count(). Using range is safer.

Fix https://github.com/element-hq/element-x-android-rageshakes/issues/1301
This commit is contained in:
Benoit Marty 2024-01-08 11:26:50 +01:00
parent 0a2df61354
commit 08d78740ab

View file

@ -247,7 +247,8 @@ class TimelinePresenter @AssistedInject constructor(
}
private fun getLastEventIdBeforeOrAt(index: Int, items: ImmutableList<TimelineItem>): EventId? {
for (item in items.subList(index, items.count())) {
for (i in index..items.count()) {
val item = items[i]
if (item is TimelineItem.Event) {
return item.eventId
}