From 3586c8d88eff0ab1ca34c6aea1514087ccaea8ef Mon Sep 17 00:00:00 2001 From: Ade Attwood Date: Tue, 23 Jul 2024 22:10:50 +0100 Subject: [PATCH] chore: clippy suggestion replace map with cloned Summary: This is a clippy suggestion. > consider calling the dedicated `cloned` method Test Plan: This has been tested locally Reviewers: AdeAttwood Reviewed By: AdeAttwood Differential Revision: https://ph.baln.co.uk/D6 --- src/config.rs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/config.rs b/src/config.rs index 1e31f10..678d8a1 100644 --- a/src/config.rs +++ b/src/config.rs @@ -65,21 +65,18 @@ impl Config { self.imap_stores .iter() .find(|store| format!(":{}:", store.name) == name) - .map(|store| store.clone()) + .cloned() } pub fn find_channel(&self, name: &str) -> Option { self.channels .iter() .find(|channel| channel.name == name) - .map(|channel| channel.clone()) + .cloned() } pub fn find_group(&self, name: &str) -> Option { - self.groups - .iter() - .find(|group| group.name == name) - .map(|group| group.clone()) + self.groups.iter().find(|group| group.name == name).cloned() } }