From 1fb658a04b4ddf143a63507443d1e8c014bc8ee0 Mon Sep 17 00:00:00 2001 From: "C. Titus Brown" Date: Fri, 23 Aug 2024 15:38:52 -0700 Subject: [PATCH] ok --- src/core/src/collection.rs | 2 +- src/core/src/manifest.rs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/core/src/collection.rs b/src/core/src/collection.rs index ef37fe3ce..b89762442 100644 --- a/src/core/src/collection.rs +++ b/src/core/src/collection.rs @@ -217,7 +217,7 @@ impl Collection { Ok(sig) } - pub fn select_picklist(&self, pick: HashSet<(&str, &str)>) -> Self { + pub fn select_picklist(&self, pick: HashSet<(String, String)>) -> Self { // @CTB: why do we need this clone here? let manifest = self.manifest.clone().select_picklist(pick); Self { manifest, storage: self.storage.clone() } diff --git a/src/core/src/manifest.rs b/src/core/src/manifest.rs index eea5fd5bb..9a52396ce 100644 --- a/src/core/src/manifest.rs +++ b/src/core/src/manifest.rs @@ -215,9 +215,9 @@ impl Manifest { self.records.len() } - pub fn select_picklist(self, pick: HashSet<(&str, &str)>) -> Self { + pub fn select_picklist(self, pick: HashSet<(String, String)>) -> Self { let records = self.records.iter().filter(|row| { - pick.contains(&(row.name().as_str(), row.md5().as_str())) + pick.contains(&(row.name().clone(), row.md5().clone())) }).cloned().collect(); Self { records }