ok

sourmash-bio · Aug 23, 2024 · 1fb658a · 1fb658a
1 parent 315dfff
commit 1fb658a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git 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
@@ -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 }