Use try operator for Option more

merge-requests/104/head
Jonas Platte 3 years ago
parent f3e630c064
commit b291e76572
No known key found for this signature in database
GPG Key ID: CC154DE0E30B7C67

@ -97,12 +97,7 @@ pub async fn get_alias_helper(
.map_or_else(Vec::new, |aliases| { .map_or_else(Vec::new, |aliases| {
aliases aliases
.iter() .iter()
.filter_map(|aliases| { .filter_map(|aliases| Regex::new(aliases.get("regex")?.as_str()?).ok())
aliases
.get("regex")
.and_then(|regex| regex.as_str())
.and_then(|regex| Regex::new(regex).ok())
})
.collect::<Vec<_>>() .collect::<Vec<_>>()
}); });

@ -1361,12 +1361,7 @@ impl Rooms {
.map_or_else(Vec::new, |users| { .map_or_else(Vec::new, |users| {
users users
.iter() .iter()
.filter_map(|users| { .filter_map(|users| Regex::new(users.get("regex")?.as_str()?).ok())
users
.get("regex")
.and_then(|regex| regex.as_str())
.and_then(|regex| Regex::new(regex).ok())
})
.collect::<Vec<_>>() .collect::<Vec<_>>()
}); });
let aliases = namespaces let aliases = namespaces
@ -1375,12 +1370,7 @@ impl Rooms {
.map_or_else(Vec::new, |aliases| { .map_or_else(Vec::new, |aliases| {
aliases aliases
.iter() .iter()
.filter_map(|aliases| { .filter_map(|aliases| Regex::new(aliases.get("regex")?.as_str()?).ok())
aliases
.get("regex")
.and_then(|regex| regex.as_str())
.and_then(|regex| Regex::new(regex).ok())
})
.collect::<Vec<_>>() .collect::<Vec<_>>()
}); });
let rooms = namespaces let rooms = namespaces

@ -1691,12 +1691,7 @@ pub(crate) fn append_incoming_pdu(
.map_or_else(Vec::new, |users| { .map_or_else(Vec::new, |users| {
users users
.iter() .iter()
.filter_map(|users| { .filter_map(|users| Regex::new(users.get("regex")?.as_str()?).ok())
users
.get("regex")
.and_then(|regex| regex.as_str())
.and_then(|regex| Regex::new(regex).ok())
})
.collect::<Vec<_>>() .collect::<Vec<_>>()
}); });
let aliases = namespaces let aliases = namespaces

Loading…
Cancel
Save