chore: prepare for rustc version bump to v1.95.0

packages/canlog/src/lib.rs

@@ -204,11 +204,11 @@ where
     }
 
     fn sort_asc(&mut self) {
-        self.entries.sort_by(|a, b| a.timestamp.cmp(&b.timestamp));
+        self.entries.sort_by_key(|a| a.timestamp);
     }
 
     fn sort_desc(&mut self) {
-        self.entries.sort_by(|a, b| b.timestamp.cmp(&a.timestamp));
+        self.entries.sort_by_key(|b| std::cmp::Reverse(b.timestamp));
     }
 }
 

rs/bitcoin/adapter/src/blockchainstate.rs

@@ -586,7 +586,7 @@ mod test {
         );
 
         let mut tips = crate::header_cache::test::get_tips(&state.header_cache);
-        tips.sort_by(|x, y| y.work.cmp(&x.work));
+        tips.sort_by_key(|y| std::cmp::Reverse(y.work));
         assert_eq!(tips.len(), 2);
         assert_eq!(tips[0].header.block_hash(), *last_fork_hash);
         assert_eq!(tips[1].header.block_hash(), *last_chain_hash);

rs/bitcoin/adapter/src/rpc_server.rs

@@ -31,6 +31,7 @@ struct BtcServiceImpl<Network: BlockchainNetwork> {
 impl TryFrom<BtcServiceGetSuccessorsRequest> for GetSuccessorsRequest {
     type Error = Status;
 
+    #[allow(clippy::result_large_err)]
     fn try_from(request: BtcServiceGetSuccessorsRequest) -> Result<Self, Self::Error> {
         let anchor = BlockHash::from_slice(request.anchor.as_slice())
             .map_err(|_| Status::unknown("Failed to parse anchor hash!"))?;

rs/bitcoin/ckbtc/minter/src/lib.rs

@@ -305,7 +305,7 @@ fn reimburse_canceled_requests<R: CanisterRuntime>(
         requests.len(),
         state.retrieve_btc_min_amount
     );
-    for (request, fee) in requests.into_iter().zip(fees.into_iter()) {
+    for (request, fee) in requests.into_iter().zip(fees) {
         if let Some(account) = request.reimbursement_account {
             let amount = request.amount.saturating_sub(fee);
             if amount > 0 {

rs/bitcoin/ckbtc/minter/tests/tests.rs

@@ -1503,7 +1503,7 @@ impl CkBtcSetup {
     pub fn print_minter_canister_logs(&self) {
         let log = self.env.canister_log(self.minter_id);
         let mut records = log.records().iter().collect::<Vec<_>>();
-        records.sort_by(|a, b| a.idx.cmp(&b.idx));
+        records.sort_by_key(|a| a.idx);
         for entry in records {
             println!(
                 "[{}] {}: {}",

rs/consensus/certification/src/certifier.rs

@@ -1044,7 +1044,7 @@ mod tests {
                     .collect();
 
                 // sort by heights
-                certs.sort_by(|s1, s2| s1.height.cmp(&s2.height));
+                certs.sort_by_key(|s| s.height);
 
                 assert_eq!(certs[0].height, Height::from(3));
                 assert_eq!(certs[1].height, Height::from(4));

rs/consensus/idkg/src/test_utils.rs

@@ -765,8 +765,8 @@ pub(crate) fn get_dealings_and_support(
 ) -> (BTreeMap<NodeId, SignedIDkgDealing>, Vec<IDkgDealingSupport>) {
     let dealings = env.nodes.create_and_verify_signed_dealings(params);
     let supports = dealings
-        .iter()
-        .flat_map(|(_, dealing)| {
+        .values()
+        .flat_map(|dealing| {
             env.nodes.filter_by_receivers(&params).map(|signer| {
                 let c: Arc<dyn ConsensusCrypto> = signer.crypto();
                 let sig_share = c

rs/crypto/internal/crypto_lib/threshold_sig/bls12_381/src/ni_dkg/fs_ni_dkg/nizk_chunking.rs

@@ -295,9 +295,8 @@ pub fn prove_chunking<R: RngCore + CryptoRng>(
     let xpowers = Scalar::xpowers(&second_challenge, NUM_ZK_REPETITIONS);
 
     let mut z_r = Vec::with_capacity(first_challenge.len());
-    let mut delta_idx = 1;
 
-    for e_i in first_challenge.iter() {
+    for (delta_idx, e_i) in (1_usize..).zip(first_challenge.iter()) {
         let mut xpow_e_ij = Vec::with_capacity(e_i.len());
         for j in 0..e_i.len() {
             xpow_e_ij.push(Scalar::muln_usize_vartime(&xpowers, &e_i[j]));
@@ -306,8 +305,6 @@ pub fn prove_chunking<R: RngCore + CryptoRng>(
         let z_rk = Scalar::muln_vartime(&witness.scalars_r, &xpow_e_ij) + &delta[delta_idx];
 
         z_r.push(z_rk);
-
-        delta_idx += 1;
     }
 
     let z_beta = Scalar::muln_vartime(&beta, &xpowers) + &delta[0];

rs/crypto/tests/canister_threshold_idkg.rs

@@ -3665,7 +3665,7 @@ mod reshare_key_transcript {
 
             let nodes_involved_in_resharing: Nodes = source_subnet_nodes
                 .into_filtered_by_receivers(&source_receivers)
-                .chain(target_subnet_nodes.into_iter())
+                .chain(target_subnet_nodes)
                 .collect();
             let initial_dealings = {
                 let signed_dealings = nodes_involved_in_resharing
@@ -3751,7 +3751,7 @@ mod reshare_key_transcript {
 
             let nodes_involved_in_resharing: Nodes = source_subnet_nodes
                 .into_filtered_by_receivers(&source_receivers)
-                .chain(target_subnet_nodes.into_iter())
+                .chain(target_subnet_nodes)
                 .collect();
             let reshared_key_transcript = nodes_involved_in_resharing
                 .run_idkg_and_create_and_verify_transcript(&reshare_params, rng);
@@ -3807,7 +3807,7 @@ mod reshare_key_transcript {
 
             let nodes_involved_in_resharing: Nodes = source_subnet_nodes
                 .into_filtered_by_receivers(&source_receivers)
-                .chain(target_subnet_nodes.into_iter())
+                .chain(target_subnet_nodes)
                 .collect();
             let reshared_key_transcript = nodes_involved_in_resharing
                 .run_idkg_and_create_and_verify_transcript(&reshare_params, rng);

rs/ethereum/cketh/minter/src/logs.rs

@@ -135,11 +135,11 @@ impl Log {
     }
 
     pub fn sort_asc(&mut self) {
-        self.entries.sort_by(|a, b| a.timestamp.cmp(&b.timestamp));
+        self.entries.sort_by_key(|a| a.timestamp);
     }
 
     pub fn sort_desc(&mut self) {
-        self.entries.sort_by(|a, b| b.timestamp.cmp(&a.timestamp));
+        self.entries.sort_by_key(|b| std::cmp::Reverse(b.timestamp));
     }
 }
 

rs/ethereum/cketh/test_utils/src/lib.rs

@@ -701,7 +701,7 @@ impl CkEthSetup {
         let log = self.env.canister_log(self.minter_id);
 
         let mut records = log.records().iter().collect::<Vec<_>>();
-        records.sort_by(|a, b| a.idx.cmp(&b.idx));
+        records.sort_by_key(|a| a.idx);
         records
             .into_iter()
             .map(|log| CanisterLog {

rs/ethereum/ledger-suite-orchestrator/src/logs/mod.rs

@@ -131,10 +131,10 @@ impl Log {
     }
 
     pub fn sort_asc(&mut self) {
-        self.entries.sort_by(|a, b| a.timestamp.cmp(&b.timestamp));
+        self.entries.sort_by_key(|a| a.timestamp);
     }
 
     pub fn sort_desc(&mut self) {
-        self.entries.sort_by(|a, b| b.timestamp.cmp(&a.timestamp));
+        self.entries.sort_by_key(|b| std::cmp::Reverse(b.timestamp));
     }
 }

rs/execution_environment/src/scheduler/tests/scheduling.rs

@@ -725,11 +725,9 @@ fn scheduler_can_deplete_induction_pool_given_enough_execution_rounds(
         available_messages,
         instructions_per_round / instructions_per_message,
     );
-    let required_rounds = if minimum_executed_messages != 0 {
-        available_messages / minimum_executed_messages + 1
-    } else {
-        1
-    };
+    let required_rounds = available_messages
+        .checked_div(minimum_executed_messages)
+        .map_or(1, |v| v + 1);
     for _ in 0..required_rounds {
         test.execute_round(ExecutionRoundType::OrdinaryRound);
     }

rs/https_outcalls/consensus/src/pool_manager.rs

@@ -2360,6 +2360,7 @@ pub mod test {
 
                 // `make_new_requests` will try to dispatch contexts to the adapter shim.
                 // Accept any number of `send` calls and treat them as no-ops.
+                #[allow(clippy::result_large_err)]
                 shim_mock.expect_send().returning(|_| Ok(()));
 
                 let mut sequence = Sequence::new();

rs/interfaces/mocks/src/crypto.rs

@@ -17,6 +17,8 @@
 //! `VetKdArgs<'a>` (`'_` is forbidden, named lifetimes cannot be declared, and
 //! elision does not work inside the macro).
 
+#![allow(clippy::result_large_err)]
+
 use ic_crypto_interfaces_sig_verification::BasicSigVerifierByPublicKey;
 use ic_interfaces::crypto::{
     BasicSigVerifier, BasicSigner, CheckKeysWithRegistryError, CurrentNodePublicKeysError,

rs/ledger_suite/icp/index/src/main.rs

@@ -460,6 +460,7 @@ fn append_blocks(new_blocks: Vec<EncodedBlock>) -> Result<(), String> {
     // the index of the next block that we
     // are going to append
     let mut block_index = with_blocks(|blocks| blocks.len());
+    #[allow(clippy::explicit_counter_loop)]
     for block in new_blocks {
         // append the encoded block to the block log
         with_blocks(|blocks| {

rs/ledger_suite/icp/index/tests/tests.rs

@@ -525,9 +525,8 @@ fn assert_ledger_index_parity_query_blocks_and_query_encoded_blocks(
         //  - the `created_at_time` field of the unencoded block is set to the `timestamp` field
         //    of the block
         //  - all the other fields of the blocks match
-        for (ledger_block, unencoded_ledger_block) in ledger_blocks
-            .into_iter()
-            .zip(ledger_unencoded_blocks.into_iter())
+        for (ledger_block, unencoded_ledger_block) in
+            ledger_blocks.into_iter().zip(ledger_unencoded_blocks)
         {
             if ledger_block != unencoded_ledger_block {
                 if ledger_block.transaction.created_at_time.is_none() {

rs/ledger_suite/icp/ledger/tests/tests.rs

@@ -611,7 +611,7 @@ fn assert_candid_block_equals_icp_ledger_block(
     icp_ledger_blocks: Vec<Block>,
 ) {
     assert_eq!(candid_blocks.len(), icp_ledger_blocks.len());
-    for (cb, lb) in candid_blocks.into_iter().zip(icp_ledger_blocks.into_iter()) {
+    for (cb, lb) in candid_blocks.into_iter().zip(icp_ledger_blocks) {
         assert_eq!(
             cb.parent_hash.map(|x| x.to_vec()),
             lb.parent_hash.map(|x| x.as_slice().to_vec())
@@ -1209,9 +1209,8 @@ fn test_block_transformation() {
     assert_eq!(certificate_pre_upgrade, certificate_post_upgrade);
 
     //Go through all blocks and make sure the blocks fetched before the upgrade are the same as after the upgrade
-    for (block_pre_upgrade, block_post_upgrade) in resp_pre_upgrade
-        .into_iter()
-        .zip(resp_post_upgrade.into_iter())
+    for (block_pre_upgrade, block_post_upgrade) in
+        resp_pre_upgrade.into_iter().zip(resp_post_upgrade)
     {
         assert_eq!(block_pre_upgrade, block_post_upgrade);
         assert_eq!(

rs/ledger_suite/icrc1/archive/src/main.rs

@@ -372,6 +372,7 @@ fn icrc3_get_blocks(reqs: Vec<GetBlocksRequest>) -> GetBlocksResult {
         }
         let length = length.min(max_length);
         let decoded_block_range = decode_block_range(start, length, decode_icrc1_block);
+        #[allow(clippy::explicit_counter_loop)]
         for block in decoded_block_range {
             blocks.push(BlockWithId {
                 id: id.clone(),

rs/ledger_suite/icrc1/index-ng/src/main.rs

@@ -946,6 +946,7 @@ fn append_blocks(new_blocks: Vec<GenericBlock>) -> Result<(), SyncError> {
     // the index of the next block that we
     // are going to append
     let mut block_index = with_blocks(|blocks| blocks.len());
+    #[allow(clippy::explicit_counter_loop)]
     for block in new_blocks {
         append_block(block_index, block)?;
         block_index += 1;

rs/ledger_suite/icrc1/index-ng/tests/tests.rs

@@ -358,7 +358,7 @@ fn assert_ledger_index_parity(env: &StateMachine, ledger_id: CanisterId, index_i
     for (index, (ledger_block, index_block)) in ledger_blocks
         .blocks
         .into_iter()
-        .zip(index_blocks.blocks.into_iter())
+        .zip(index_blocks.blocks)
         .enumerate()
     {
         // If the hash matches then they are the same block.

rs/ledger_suite/icrc1/ledger/tests/tests.rs

@@ -1232,7 +1232,7 @@ fn test_icrc3_get_blocks() {
     for (local_index, (actual_block, expected_block)) in actual_res
         .blocks
         .into_iter()
-        .zip(expected_res.blocks.into_iter())
+        .zip(expected_res.blocks)
         .enumerate()
     {
         check_old_vs_icrc3_blocks(
@@ -1285,7 +1285,7 @@ fn test_icrc3_get_blocks() {
     for (block_index, (actual_block, expected_block)) in actual_archived_blocks
         .blocks
         .into_iter()
-        .zip(expected_archived_blocks.blocks.into_iter())
+        .zip(expected_archived_blocks.blocks)
         .enumerate()
     {
         check_old_vs_icrc3_blocks(block_index, expected_block.clone(), actual_block.clone());

rs/ledger_suite/tests/sm-tests/src/lib.rs

@@ -1615,7 +1615,7 @@ where
     let mut prev_hash = None;
 
     // Check that the hash chain is correct.
-    for block in archived_blocks.into_iter().chain(resp.blocks.into_iter()) {
+    for block in archived_blocks.into_iter().chain(resp.blocks) {
         assert_eq!(
             prev_hash,
             get_phash(&block).expect("cannot get the hash of the previous block")
@@ -2069,7 +2069,7 @@ pub fn icrc1_test_block_transformation<T, Tokens>(
     for (block_pre_upgrade, block_post_upgrade) in resp_pre_upgrade
         .blocks
         .into_iter()
-        .zip(resp_post_upgrade.blocks.into_iter())
+        .zip(resp_post_upgrade.blocks)
     {
         assert!(
             equivalent_values(&block_pre_upgrade, &block_post_upgrade),

rs/monitoring/adapter_metrics/server/src/lib.rs

@@ -28,6 +28,7 @@ impl Metrics {
 
 #[tonic::async_trait]
 impl AdapterMetricsService for Metrics {
+    #[allow(clippy::result_large_err)]
     async fn scrape(
         &self,
         _request: Request<ScrapeRequest>,

rs/nervous_system/agent/src/sns/swap.rs

@@ -78,7 +78,7 @@ impl SwapCanister {
             if new_sns_neuron_recipes.sns_neuron_recipes.is_empty() {
                 return Ok(sns_neuron_recipes);
             } else {
-                sns_neuron_recipes.extend(new_sns_neuron_recipes.sns_neuron_recipes.into_iter())
+                sns_neuron_recipes.extend(new_sns_neuron_recipes.sns_neuron_recipes)
             }
         }
         Err(ListAllSnsNeuronRecipesError::TooManyRecipes(

rs/nervous_system/integration_tests/tests/sns_lifecycle.rs

@@ -294,7 +294,7 @@ async fn test_sns_lifecycle(
             .await
             .unwrap();
 
-        stream::iter(nns_neuron_controller_principal_ids.into_iter())
+        stream::iter(nns_neuron_controller_principal_ids)
             .then(|controller_principal_id| {
                 let pocket_ic = &pocket_ic;
                 async move {
@@ -1589,8 +1589,7 @@ async fn test_sns_lifecycle(
                             permissions: &[sns_pb::NeuronPermission],
                         ) -> Vec<sns_pb::NeuronPermission> {
                             let mut permissions = permissions.to_vec();
-                            permissions
-                                .sort_by(|x, y| y.principal.unwrap().cmp(&x.principal.unwrap()));
+                            permissions.sort_by_key(|y| std::cmp::Reverse(y.principal.unwrap()));
                             permissions
                         }
                         let claimer_permissions = nervous_system_parameters

rs/nns/cmc/src/main.rs

@@ -964,15 +964,12 @@ fn compute_average_icp_xdr_rate_at_time(
     let size = filtered_rates.len() as u64;
     // If there are rates that meet the age requirement, compute the sum and compute
     // the average.
-    if size > 0 {
-        let sum: u64 = filtered_rates.into_iter().sum();
-        Some(IcpXdrConversionRate {
-            timestamp_seconds: day * 86_400,   // Start of the current day.
-            xdr_permyriad_per_icp: sum / size, // The average of the valid data points.
+    let sum: u64 = filtered_rates.into_iter().sum();
+    sum.checked_div(size)
+        .map(|xdr_permyriad_per_icp| IcpXdrConversionRate {
+            timestamp_seconds: day * 86_400, // Start of the current day.
+            xdr_permyriad_per_icp,           // The average of the valid data points.
         })
-    } else {
-        None
-    }
 }
 
 #[update(hidden = true)]

rs/nns/governance/src/governance.rs

@@ -4660,6 +4660,7 @@ impl Governance {
                     ));
                 }
             }
+            #[allow(clippy::collapsible_match)]
             Command::Follow(follow) => {
                 if follow.followees.len() > MAX_FOLLOWEES_PER_TOPIC {
                     return Err(GovernanceError::new_with_message(
@@ -4955,6 +4956,7 @@ impl Governance {
                 Self::validate_add_or_remove_data_centers_payload(&update.payload)
                     .map_err(invalid_proposal_error)?;
             }
+            #[allow(clippy::collapsible_match)]
             ValidNnsFunction::SplitSubnet => {
                 if !are_subnet_splitting_proposals_enabled() {
                     return Err(invalid_proposal_error(String::from(

rs/nns/governance/src/governance/benches.rs

@@ -303,7 +303,7 @@ fn set_up_chain<R: Rng>(
         let previous_neuron_indices = (neuron_index - num_half_followees - 1)..neuron_index;
         let followee_neuron_ids = previous_neuron_indices
             .map(|index| neuron_ids[index as usize])
-            .chain(not_voting_neuron_ids.clone().into_iter())
+            .chain(not_voting_neuron_ids.clone())
             .collect::<Vec<_>>();
 
         let followees = hashmap! {topic.into() => Followees {followees: followee_neuron_ids}};

rs/nns/governance/tests/governance.rs

@@ -10931,8 +10931,7 @@ async fn test_known_neurons() {
         },
     ];
     let mut sorted_response_known_neurons = gov.list_known_neurons().known_neurons;
-    sorted_response_known_neurons
-        .sort_by(|a, b| a.id.as_ref().unwrap().id.cmp(&b.id.as_ref().unwrap().id));
+    sorted_response_known_neurons.sort_by_key(|a| a.id.as_ref().unwrap().id);
     assert_eq!(sorted_response_known_neurons, expected_known_neurons);
 
     // This proposal tries to name neuron 1 with the already existing name "Two", this should fail.
@@ -10959,8 +10958,7 @@ async fn test_known_neurons() {
 
     // Check that the state is the same as before the last proposal.
     let mut sorted_response_known_neurons = gov.list_known_neurons().known_neurons;
-    sorted_response_known_neurons
-        .sort_by(|a, b| a.id.as_ref().unwrap().id.cmp(&b.id.as_ref().unwrap().id));
+    sorted_response_known_neurons.sort_by_key(|a| a.id.as_ref().unwrap().id);
     assert_eq!(sorted_response_known_neurons, expected_known_neurons);
 
     // Update the name of neuron 2.