From 49f77c672ab683d35998c931db39a41075d0ab45 Mon Sep 17 00:00:00 2001 From: Josh Mitchell Date: Mon, 16 Nov 2020 14:01:42 +1100 Subject: [PATCH] Deref rather than clone in Frame::filter_coords() --- src/frame.rs | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/frame.rs b/src/frame.rs index f8b6c50..79a06e3 100644 --- a/src/frame.rs +++ b/src/frame.rs @@ -40,14 +40,13 @@ impl Frame { } /// Filters the frame by removing all atoms not matching the given indeces. - pub fn filter_coords(self: &mut Frame, indeces: &[usize]) { + pub fn filter_coords(self: &mut Frame, indices: &[usize]) { self.coords = self .coords .iter() - .map(|elem| elem.clone()) .enumerate() - .filter(|&(i, _)| indeces.contains(&i)) - .map(|(_, elem)| elem) + .filter(|(i, _)| indices.contains(i)) + .map(|(_, elem)| *elem) .collect(); }