People::get_all() does not need mutable self

This commit is contained in:
Mike Dilger 2023-01-02 21:44:35 +13:00
parent 14d18d635c
commit a85c416657

View File

@ -195,7 +195,7 @@ impl People {
} }
} }
pub fn get_all(&mut self) -> Vec<DbPerson> { pub fn get_all(&self) -> Vec<DbPerson> {
let mut v: Vec<DbPerson> = self.people.values().map(|p| p.to_owned()).collect(); let mut v: Vec<DbPerson> = self.people.values().map(|p| p.to_owned()).collect();
v.sort_by(|a, b| { v.sort_by(|a, b| {
let c = a.name.cmp(&b.name); let c = a.name.cmp(&b.name);