Lint fixes and remove TODOs

smartcontractkit · Feb 1, 2025 · 523c1d2 · 523c1d2
1 parent 1142668
commit 523c1d2
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 18 deletions.
diff --git a/chains/solana/contracts/programs/ccip-router/src/instructions/v1/onramp.rs b/chains/solana/contracts/programs/ccip-router/src/instructions/v1/onramp.rs
@@ -258,11 +258,11 @@ fn get_fee_cpi<'info>(
             .token_amounts
             .iter()
             .map(|x| fee_quoter::messages::SVMTokenAmount {
-                token: x.token.clone(),
+                token: x.token,
                 amount: x.amount,
             })
             .collect(),
-        fee_token: message.fee_token.clone(),
+        fee_token: message.fee_token,
         extra_args: message.extra_args.clone(),
     };
 
@@ -284,7 +284,7 @@ fn token_transfer(
 
     require!(
         extra_data_length <= CCIP_LOCK_OR_BURN_V1_RET_BYTES
-            || extra_data_length <= additional_data.dest_bytes_overhead, // TODO is this ok here?
+            || extra_data_length <= additional_data.dest_bytes_overhead,
         CcipRouterError::SourceTokenDataTooLarge
     );
 

diff --git a/chains/solana/contracts/programs/ccip-router/src/instructions/v1/pools.rs b/chains/solana/contracts/programs/ccip-router/src/instructions/v1/pools.rs
@@ -49,7 +49,7 @@ pub(super) struct TokenAccounts<'a> {
     pub pool_signer: &'a AccountInfo<'a>,
     pub token_program: &'a AccountInfo<'a>,
     pub mint: &'a AccountInfo<'a>,
-    pub fee_token_config: &'a AccountInfo<'a>, // TODO review this...
+    pub fee_token_config: &'a AccountInfo<'a>,
     pub remaining_accounts: &'a [AccountInfo<'a>],
 }
 

diff --git a/chains/solana/contracts/programs/ccip-router/src/state.rs b/chains/solana/contracts/programs/ccip-router/src/state.rs
@@ -16,7 +16,7 @@ pub struct Config {
 
     _padding2: [u8; 8],
     pub ocr3: [Ocr3Config; 2],
-    pub fee_quoter: Pubkey, // TODO configure this
+    pub fee_quoter: Pubkey,
 
     pub link_token_mint: Pubkey,
     pub fee_aggregator: Pubkey, // Allowed address to withdraw billed fees to (will use ATAs derived from it)
@@ -200,19 +200,6 @@ pub struct RateLimitTokenBucket {
     pub rate: u128,        // Number of tokens per second that the bucket is refilled.
 }
 
-#[derive(InitSpace, Clone, AnchorSerialize, AnchorDeserialize, Debug)]
-// TODO remove?
-pub struct BillingTokenConfig {
-    // NOTE: when modifying this struct, make sure to update the version in the wrapper
-    pub enabled: bool,
-    pub mint: Pubkey,
-
-    // price tracking
-    pub usd_per_token: TimestampedPackedU224,
-    // billing configs
-    pub premium_multiplier_wei_per_eth: u64,
-}
-
 #[derive(InitSpace, Clone, AnchorSerialize, AnchorDeserialize, Debug)]
 pub struct TimestampedPackedU224 {
     pub value: [u8; 28],