Be more efficient about how we find room members.

pkulak · May 13, 2023 · 756372f · 756372f
1 parent 60fb762
commit 756372f
Showing 1 changed file with 9 additions and 13 deletions.
diff --git a/src/widgets/chat.rs b/src/widgets/chat.rs
@@ -528,23 +528,19 @@ impl Chat {
                     members.push(member);
                 }
 
-                if members.len() > 5 {
+                if members.len() > 5 || members.len() == self.members.len() {
                     break;
                 }
             }
 
-            let names: Vec<&str> = members
-                .iter()
-                .map(|m| {
-                    m.display_name()
-                        .unwrap_or_else(|| m.user_id().localpart())
-                        .split_whitespace()
-                        .next()
-                        .unwrap_or_default()
-                })
-                .collect();
-
-            let iter = names.into_iter().map(|n| n.to_string());
+            let iter = members.iter().map(|m| {
+                m.display_name()
+                    .unwrap_or_else(|| m.user_id().localpart())
+                    .split_whitespace()
+                    .next()
+                    .unwrap_or_default()
+                    .to_string()
+            });
 
             pretty_list(limit_list(iter, 5, self.members.len(), Some("at least")))
         })