fix: remove unnecessary clones

Author: hstove

stacks-signer/src/lib.rs

@@ -147,7 +147,7 @@ impl<S: Signer<T> + Send + 'static, T: SignerEventTrait + 'static> SpawnedSigner
             cmd_send,
             res_recv,
             _phantom: std::marker::PhantomData,
-            config: config.clone(),
+            config,
         }
     }
 }

testnet/stacks-node/src/tests/nakamoto_integrations.rs

@@ -105,7 +105,7 @@ use crate::tests::neon_integrations::{
 };
 use crate::tests::{
     get_chain_info, make_contract_publish, make_contract_publish_versioned, make_stacks_transfer,
-    set_random_binds, to_addr,
+    to_addr,
 };
 use crate::{tests, BitcoinRegtestController, BurnchainController, Config, ConfigFile, Keychain};
 

testnet/stacks-node/src/tests/signer/v0.rs

@@ -2393,7 +2393,7 @@ fn signer_set_rollover() {
     let short_timeout = Duration::from_secs(20);
 
     // Verify that naka_conf has our new signer's event observers
-    for toml in new_signer_configs.clone() {
+    for toml in &new_signer_configs {
         let signer_config = SignerConfig::load_from_str(&toml).unwrap();
         let endpoint = format!("{}", signer_config.endpoint);
         assert!(signer_test