fix: avoid cloning double reference & remove unnecessary clone

stackslib/src/chainstate/burn/db/sortdb.rs

@@ -5580,9 +5580,7 @@ impl<'a> SortitionHandleTx<'a> {
         hash_tied.sort();
         let winner = hash_tied
             .first()
-            .as_ref()
-            .expect("FATAL: zero-length list of tied block IDs")
-            .clone();
+            .expect("FATAL: zero-length list of tied block IDs");
         let winner_index = *mapping
             .get(&winner)
             .expect("FATAL: winning block ID not mapped");

stackslib/src/chainstate/coordinator/tests.rs

@@ -360,7 +360,7 @@ pub fn setup_states_with_epochs(
                 burnchain_blocks_db,
                 first_sortition.burn_header_hash,
                 registers,
-                path.clone(),
+                *path,
             ));
         } else {
             others.push(burnchain_blocks_db);

stackslib/src/chainstate/stacks/tests/mod.rs

@@ -254,7 +254,7 @@ impl TestMinerTrace {
         let mut all_test_names = HashSet::new();
         for p in self.points.iter() {
             for miner_id in p.miner_node_map.keys() {
-                if let Some(ref test_name) = p.miner_node_map.get(miner_id) {
+                if let Some(test_name) = p.miner_node_map.get(miner_id) {
                     if !all_test_names.contains(test_name) {
                         all_test_names.insert(test_name.clone());
                     }

stackslib/src/clarity_vm/database/marf.rs

@@ -88,7 +88,7 @@ impl MarfedKV {
     ) -> InterpreterResult<MarfedKV> {
         let marf = MarfedKV::setup_db(path_str, false, marf_opts)?;
         let chain_tip = match miner_tip {
-            Some(ref miner_tip) => *miner_tip.clone(),
+            Some(miner_tip) => miner_tip.clone(),
             None => StacksBlockId::sentinel(),
         };
 
@@ -102,7 +102,7 @@ impl MarfedKV {
     ) -> InterpreterResult<MarfedKV> {
         let marf = MarfedKV::setup_db(path_str, true, marf_opts)?;
         let chain_tip = match miner_tip {
-            Some(ref miner_tip) => *miner_tip.clone(),
+            Some(miner_tip) => miner_tip.clone(),
             None => StacksBlockId::sentinel(),
         };
 

stackslib/src/net/chat.rs

@@ -718,7 +718,7 @@ impl ConversationP2P {
         chain_view: &BurnchainView,
     ) -> bool {
         let bhh = match chain_view.last_burn_block_hashes.get(&block_height) {
-            Some(ref bhh) => bhh.clone(),
+            Some(bhh) => bhh,
             None => {
                 // not present; can't prove disagreement (assume the remote peer is just stale)
                 return false;

stackslib/src/net/neighbors/walk.rs

@@ -442,7 +442,7 @@ impl<DB: NeighborWalkDB, NC: NeighborComms> NeighborWalk<DB, NC> {
         );
 
         let (addr, pingback_peer) = match network.get_walk_pingbacks().iter().skip(idx).next() {
-            Some((ref addr, ref pingback_peer)) => (addr.clone(), pingback_peer.clone()),
+            Some((addr, pingback_peer)) => (addr, pingback_peer),
             None => {
                 return Err(net_error::NoSuchNeighbor);
             }