chore: remove commented-out code in sdk (#1551)

Author: sergeytimoshin

sdk-libs/sdk/src/address.rs

@@ -9,22 +9,6 @@ use solana_program::account_info::AccountInfo;
 
 use crate::merkle_context::{AddressMerkleContext, RemainingAccounts};
 
-// #[derive(Debug, PartialEq, Default, Clone, AnchorDeserialize, AnchorSerialize)]
-// pub struct NewAddressParams {
-//     pub seed: [u8; 32],
-//     pub address_queue_pubkey: Pubkey,
-//     pub address_merkle_tree_pubkey: Pubkey,
-//     pub address_merkle_tree_root_index: u16,
-// }
-
-// #[derive(Debug, PartialEq, Default, Clone, Copy, AnchorDeserialize, AnchorSerialize)]
-// pub struct PackedNewAddressParams {
-//     pub seed: [u8; 32],
-//     pub address_queue_account_index: u8,
-//     pub address_merkle_tree_account_index: u8,
-//     pub address_merkle_tree_root_index: u16,
-// }
-
 pub struct AddressWithMerkleContext {
     pub address: [u8; 32],
     pub address_merkle_context: AddressMerkleContext,

sdk-libs/sdk/src/compressed_account.rs

@@ -5,12 +5,10 @@ pub mod account;
 pub mod account_info;
 pub mod account_meta;
 pub mod address;
-// pub mod compressed_account;
 pub mod constants;
 pub use constants::*;
 pub mod context;
 pub mod error;
-// pub mod event;
 pub mod instruction_data;
 pub mod legacy;
 pub mod merkle_context;

sdk-libs/sdk/src/event.rs

@@ -54,26 +54,6 @@ impl RemainingAccounts {
     }
 }
 
-// #[derive(Debug, Clone, Copy, AnchorSerialize, AnchorDeserialize, PartialEq, Default)]
-// pub struct MerkleContext {
-//     pub merkle_tree_pubkey: Pubkey,
-//     pub nullifier_queue_pubkey: Pubkey,
-//     pub leaf_index: u32,
-//     /// Index of leaf in queue. Placeholder of batched Merkle tree updates
-//     /// currently unimplemented.
-//     pub prove_by_index: bool,
-// }
-
-// #[derive(Debug, Clone, Copy, AnchorSerialize, AnchorDeserialize, PartialEq, Default)]
-// pub struct PackedMerkleContext {
-//     pub merkle_tree_pubkey_index: u8,
-//     pub nullifier_queue_pubkey_index: u8,
-//     pub leaf_index: u32,
-//     /// Index of leaf in queue. Placeholder of batched Merkle tree updates
-//     /// currently unimplemented.
-//     pub prove_by_index: bool,
-// }
-
 pub fn pack_merkle_contexts<'a, I>(
     merkle_contexts: I,
     remaining_accounts: &'a mut RemainingAccounts,

sdk-libs/sdk/src/lib.rs

@@ -29,23 +29,6 @@ pub struct NewAddressProofWithContext {
     pub new_element_next_value: Option<BigUint>,
 }
 
-// #[derive(Debug, Clone, PartialEq, Eq, AnchorDeserialize, AnchorSerialize)]
-// pub struct CompressedProof {
-//     pub a: [u8; 32],
-//     pub b: [u8; 64],
-//     pub c: [u8; 32],
-// }
-
-// impl Default for CompressedProof {
-//     fn default() -> Self {
-//         Self {
-//             a: [0; 32],
-//             b: [0; 64],
-//             c: [0; 32],
-//         }
-//     }
-// }
-
 #[derive(Debug, AnchorDeserialize, AnchorSerialize)]
 pub struct ProofRpcResult {
     pub proof: CompressedProof,

sdk-libs/sdk/src/merkle_context.rs

@@ -20,31 +20,6 @@ pub fn find_cpi_signer(program_id: &Pubkey) -> Pubkey {
     Pubkey::find_program_address([CPI_AUTHORITY_PDA_SEED].as_slice(), program_id).0
 }
 
-// #[derive(AnchorSerialize, AnchorDeserialize, Debug, Clone, Copy, PartialEq, Eq, Default)]
-// pub struct CompressedCpiContext {
-//     /// Is set by the program that is invoking the CPI to signal that is should
-//     /// set the cpi context.
-//     pub set_context: bool,
-//     /// Is set to clear the cpi context since someone could have set it before
-//     /// with unrelated data.
-//     pub first_set_context: bool,
-//     /// Index of cpi context account in remaining accounts.
-//     pub cpi_context_account_index: u8,
-// }
-
-// #[derive(Debug, PartialEq, Default, Clone, AnchorDeserialize, AnchorSerialize)]
-// pub struct InstructionDataInvokeCpi {
-//     pub proof: Option<CompressedProof>,
-//     pub new_address_params: Vec<PackedNewAddressParams>,
-//     pub input_compressed_accounts_with_merkle_context:
-//         Vec<PackedCompressedAccountWithMerkleContext>,
-//     pub output_compressed_accounts: Vec<OutputCompressedAccountWithPackedContext>,
-//     pub relay_fee: Option<u64>,
-//     pub compress_or_decompress_lamports: Option<u64>,
-//     pub is_compress: bool,
-//     pub cpi_context: Option<CompressedCpiContext>,
-// }
-
 #[inline(always)]
 pub fn setup_cpi_accounts<'info>(
     ctx: &Context<