v1.18: sdk: Rederive Copy on Rent and EpochSchedule (#1744)

sdk: Rederive Clone on Rent and EpochSchedule

Author: joncinque

accounts-db/src/accounts_db.rs

@@ -8997,8 +8997,10 @@ impl AccountsDb {
         let schedule = &genesis_config.epoch_schedule;
         let rent_collector = RentCollector::new(
             schedule.get_epoch(max_slot),
+            #[allow(clippy::clone_on_copy)]
             schedule.clone(),
             genesis_config.slots_per_year(),
+            #[allow(clippy::clone_on_copy)]
             genesis_config.rent.clone(),
         );
         let accounts_data_len = AtomicU64::new(0);

core/src/repair/ancestor_hashes_service.rs

@@ -1349,6 +1349,7 @@ mod test {
         fn new(bank_forks: Arc<RwLock<BankForks>>) -> Self {
             let ancestor_hashes_request_statuses = Arc::new(DashMap::new());
             let ancestor_hashes_request_socket = Arc::new(UdpSocket::bind("0.0.0.0:0").unwrap());
+            #[allow(clippy::clone_on_copy)]
             let epoch_schedule = bank_forks
                 .read()
                 .unwrap()

core/src/repair/repair_weight.rs

@@ -2553,6 +2553,7 @@ mod test {
         let stake = 100;
         let (bank, vote_pubkeys) = bank_utils::setup_bank_and_vote_pubkeys_for_tests(10, stake);
         let mut epoch_stakes = bank.epoch_stakes_map().clone();
+        #[allow(clippy::clone_on_copy)]
         let mut epoch_schedule = bank.epoch_schedule().clone();
 
         // Simulate epoch boundary at slot 10, where half of the stake deactivates

core/src/shred_fetch_stage.rs

@@ -36,6 +36,7 @@ pub(crate) struct ShredFetchStage {
 
 impl ShredFetchStage {
     // updates packets received on a channel and sends them on another channel
+    #[allow(clippy::clone_on_copy)]
     fn modify_packets(
         recvr: PacketBatchReceiver,
         sendr: Sender<PacketBatch>,

core/src/tvu.rs

@@ -200,6 +200,7 @@ impl Tvu {
         let (dumped_slots_sender, dumped_slots_receiver) = unbounded();
         let (popular_pruned_forks_sender, popular_pruned_forks_receiver) = unbounded();
         let window_service = {
+            #[allow(clippy::clone_on_copy)]
             let epoch_schedule = bank_forks
                 .read()
                 .unwrap()

genesis/src/stakes.rs

@@ -246,6 +246,7 @@ mod tests {
         let total_lamports = staker_reserve + reserve * 2 + 1;
         create_and_check_stakes(
             &mut GenesisConfig {
+                #[allow(clippy::clone_on_copy)]
                 rent: rent.clone(),
                 ..GenesisConfig::default()
             },
@@ -272,6 +273,7 @@ mod tests {
         let total_lamports = staker_reserve + reserve * 2 + 1;
         create_and_check_stakes(
             &mut GenesisConfig {
+                #[allow(clippy::clone_on_copy)]
                 rent: rent.clone(),
                 ..GenesisConfig::default()
             },
@@ -298,6 +300,7 @@ mod tests {
         let total_lamports = staker_reserve + (granularity + reserve) * 2;
         create_and_check_stakes(
             &mut GenesisConfig {
+                #[allow(clippy::clone_on_copy)]
                 rent: rent.clone(),
                 ..GenesisConfig::default()
             },
@@ -323,6 +326,7 @@ mod tests {
         let total_lamports = staker_reserve + (granularity + reserve + 1) * 2;
         create_and_check_stakes(
             &mut GenesisConfig {
+                #[allow(clippy::clone_on_copy)]
                 rent: rent.clone(),
                 ..GenesisConfig::default()
             },

ledger/src/blockstore_processor.rs

@@ -3887,6 +3887,7 @@ pub mod tests {
             AccountSecondaryIndexes::default(),
             AccountShrinkThreshold::default(),
         );
+        #[allow(clippy::clone_on_copy)]
         bank.epoch_schedule().clone()
     }
 

ledger/src/leader_schedule_cache.rs

@@ -40,6 +40,7 @@ pub struct LeaderScheduleCache {
 
 impl LeaderScheduleCache {
     pub fn new_from_bank(bank: &Bank) -> Self {
+        #[allow(clippy::clone_on_copy)]
         Self::new(bank.epoch_schedule().clone(), bank)
     }
 

programs/bpf_loader/src/syscalls/mod.rs

@@ -3362,8 +3362,10 @@ mod tests {
 
         let mut sysvar_cache = SysvarCache::default();
         sysvar_cache.set_clock(src_clock.clone());
+        #[allow(clippy::clone_on_copy)]
         sysvar_cache.set_epoch_schedule(src_epochschedule.clone());
         sysvar_cache.set_fees(src_fees.clone());
+        #[allow(clippy::clone_on_copy)]
         sysvar_cache.set_rent(src_rent.clone());
         sysvar_cache.set_epoch_rewards(src_rewards);
 

programs/vote/src/vote_state/mod.rs

@@ -1298,6 +1298,7 @@ mod tests {
         let processor_account = AccountSharedData::new(0, 0, &solana_sdk::native_loader::id());
         let transaction_context = TransactionContext::new(
             vec![(id(), processor_account), (node_pubkey, vote_account)],
+            #[allow(clippy::clone_on_copy)]
             rent.clone(),
             0,
             0,