clippy: unnecessary vecs

ElementsProject · Aug 8, 2024 · 97b2229 · 97b2229
1 parent 8894215
commit 97b2229
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/examples/sign_multisig.rs b/examples/sign_multisig.rs
@@ -47,7 +47,7 @@ fn main() {
     };
 
     #[rustfmt::skip]
-    let public_keys = vec![
+    let public_keys = [
         bitcoin::PublicKey::from_slice(&[2; 33]).expect("key 1"),
         bitcoin::PublicKey::from_slice(&[
             0x02,

diff --git a/src/policy/concrete.rs b/src/policy/concrete.rs
@@ -1264,7 +1264,7 @@ fn with_huffman_tree<Pk: MiniscriptKey>(
 /// any one of the conditions exclusively.
 #[cfg(feature = "compiler")]
 fn generate_combination<Pk: MiniscriptKey>(
-    policy_vec: &Vec<Arc<PolicyArc<Pk>>>,
+    policy_vec: &[Arc<PolicyArc<Pk>>],
     prob: f64,
     k: usize,
 ) -> Vec<(f64, Arc<PolicyArc<Pk>>)> {

diff --git a/src/psbt/mod.rs b/src/psbt/mod.rs
@@ -443,6 +443,7 @@ impl<'psbt, Pk: MiniscriptKey + ToPublicKey> Satisfier<Pk> for PsbtInputSatisfie
     }
 }
 
+#[allow(clippy::ptr_arg)] // complains about &Vec but this is used in a closure context
 fn try_vec_as_preimage32(vec: &Vec<u8>) -> Option<Preimage32> {
     if vec.len() == 32 {
         let mut arr = [0u8; 32];