Lint

chainwayxyz · Dec 18, 2024 · 0b69f36 · 0b69f36
1 parent 487c84a
commit 0b69f36
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 8 deletions.
diff --git a/bin/citrea/src/rollup/mod.rs b/bin/citrea/src/rollup/mod.rs
@@ -52,7 +52,7 @@ pub trait CitreaRollupBlueprint: RollupBlueprint {
         <Self::NativeContext as Spec>::Storage: NativeStorage,
     {
         // Set forks before anything
-        set_forks(rollup_config.forks.clone());
+        set_forks(rollup_config.forks);
 
         let mut task_manager = TaskManager::default();
         let da_service = self
@@ -176,7 +176,7 @@ pub trait CitreaRollupBlueprint: RollupBlueprint {
         <Self::NativeContext as Spec>::Storage: NativeStorage,
     {
         // Set forks before anything
-        set_forks(rollup_config.forks.clone());
+        set_forks(rollup_config.forks);
 
         let mut task_manager = TaskManager::default();
         let da_service = self
@@ -311,7 +311,7 @@ pub trait CitreaRollupBlueprint: RollupBlueprint {
         <Self::NativeContext as Spec>::Storage: NativeStorage,
     {
         // Set forks before anything
-        set_forks(rollup_config.forks.clone());
+        set_forks(rollup_config.forks);
 
         let mut task_manager = TaskManager::default();
         let da_service = self
@@ -445,7 +445,7 @@ pub trait CitreaRollupBlueprint: RollupBlueprint {
         <Self::NativeContext as Spec>::Storage: NativeStorage,
     {
         // Set forks before anything
-        set_forks(rollup_config.forks.clone());
+        set_forks(rollup_config.forks);
 
         // Migrate before constructing ledger_db instance so that no lock is present.
         let migrator = LedgerDBMigrator::new(

diff --git a/crates/common/src/config.rs b/crates/common/src/config.rs
@@ -306,7 +306,7 @@ impl FromEnv for LightClientProverConfig {
 impl FromEnv for Forks {
     fn from_env() -> anyhow::Result<Self> {
         let forks = std::env::var("FORKS")?;
-        Ok(Forks::from_utf8(&forks).ok_or(anyhow::anyhow!("Invalid FORKS env variable"))?)
+        Forks::from_utf8(&forks).ok_or(anyhow::anyhow!("Invalid FORKS env variable"))
     }
 }
 

diff --git a/crates/prover-services/src/parallel/mod.rs b/crates/prover-services/src/parallel/mod.rs
@@ -49,6 +49,7 @@ where
     Stf::PreState: Clone + Send + Sync,
 {
     /// Creates a new prover.
+    #[allow(clippy::too_many_arguments)]
     pub fn new(
         da_service: Arc<Da>,
         vm: Vm,

diff --git a/crates/sovereign-sdk/rollup-interface/src/fork/mod.rs b/crates/sovereign-sdk/rollup-interface/src/fork/mod.rs
@@ -330,9 +330,9 @@ pub(crate) const fn verify_forks(forks: &[Fork]) -> bool {
 /// This assumes that the list of forks is sorted by block number in ascending fashion.
 pub(crate) fn fork_pos_from_block_number(forks: &[Fork], block_number: u64) -> usize {
     let pos = forks.binary_search_by(|fork| fork.activation_height.cmp(&block_number));
-    let active_fork_idx = match pos {
+
+    match pos {
         Ok(idx) => idx,
         Err(idx) => idx.saturating_sub(1),
-    };
-    active_fork_idx
+    }
 }