refactor: Try to avoid filtering all event items before finding one with the wanted id

2025-11-04 12:18:03 +01:00
parent a1768ea518
commit 8b805b1ea5
1 changed files with 6 additions and 3 deletions
@@ -449,9 +449,12 @@ impl ReadReceiptTimelineUpdate {
                // Don't iterate over all items if the `old_item_pos` is known: the `item_pos`
                // for the new item is necessarily _after_ the old item.
                .skip_while(|(nth, _)| *nth < old_item_pos)
-                .filter_map(|(nth, item)| Some((nth, item.as_event()?)))
-                .find_map(|(nth, event_item)| {
-                    (event_item.event_id() == Some(&event_id)).then_some(nth)
+                .find_map(|(nth, item)| {
+                    if let Some(event_item) = item.as_event() {
+                        (event_item.event_id() == Some(&event_id)).then_some(nth)
+                    } else {
+                        None
+                    }
                })
        });