rewrite is_none_or_empty to not use clone()

ElementsProject · Nov 14, 2023 · 6a50b17 · 6a50b17
1 parent 02ca226
commit 6a50b17
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/cln-rpc/src/lib.rs b/cln-rpc/src/lib.rs
@@ -130,9 +130,11 @@ fn is_none_or_empty<T>(f: &Option<Vec<T>>) -> bool
 where
     T: Clone,
 {
-    // TODO Find a better way to check, possibly without cloning
-    let f = f.clone();
-    f.is_none() || f.unwrap().is_empty()
+    if let Some(inner) = f {
+        inner.is_empty()
+    } else {
+        true
+    }
 }
 
 #[cfg(test)]