fix: remove unnecessary comments (#691)

* fix: remove comments

* chore: pass build

* chore: pass build

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>

Author: ibrizsabin

contracts/cosmwasm-vm/cw-ibc-core/src/ics02_client/client.rs

@@ -399,7 +399,6 @@ impl<'a> CwIbcCoreContext<'a> {
     }
 }
 
-//TODO : Implement Methods
 #[allow(dead_code)]
 #[allow(unused_variables)]
 impl<'a> CwIbcCoreContext<'a> {

contracts/cosmwasm-vm/cw-ibc-core/src/ics02_client/handler.rs

@@ -194,7 +194,6 @@ impl<'a> IbcClient for CwIbcCoreContext<'a> {
                 error: ClientError::MissingRawConsensusState,
             })?;
 
-        //Check Client Frozen
         if old_client_state.is_frozen() {
             return Err(ClientError::ClientFrozen {
                 client_id: client_id.clone(),
@@ -213,7 +212,7 @@ impl<'a> IbcClient for CwIbcCoreContext<'a> {
                 time2: now,
             })
             .map_err(Into::<ContractError>::into)?;
-        // Check if the latest consensus state is within the trust period.
+
         if old_client_state.expired(duration) {
             return Err(ClientError::HeaderNotWithinTrustPeriod {
                 latest_time: old_consensus_state.timestamp(),
@@ -222,8 +221,6 @@ impl<'a> IbcClient for CwIbcCoreContext<'a> {
             .map_err(Into::<ContractError>::into);
         };
 
-        // Validate the upgraded client state and consensus state and verify proofs against the root
-
         let wasm_exec_message = LightClientMessage::UpgradeClient {
             upgraded_client_state: new_client_state.value,
             upgraded_consensus_state: new_consensus_state.value,

contracts/cosmwasm-vm/cw-ibc-core/src/ics03_connection/connection.rs

@@ -305,14 +305,11 @@ impl<'a> CwIbcCoreContext<'a> {
     }
 }
 
-//TODO : Implement Methods
-#[allow(dead_code)]
-#[allow(unused_variables)]
 impl<'a> CwIbcCoreContext<'a> {
     pub fn commitment_prefix(&self, deps: Deps, env: &Env) -> CommitmentPrefix {
         let address = self.get_self_address(deps, env);
         let prefix = get_address_storage_prefix(&address, StorageKey::Commitments.as_str());
-        CommitmentPrefix::try_from(prefix).unwrap_or_default() //TODO
+        CommitmentPrefix::try_from(prefix).unwrap_or_default()
     }
 
     fn get_self_address(&self, deps: Deps, env: &Env) -> String {

contracts/cosmwasm-vm/cw-ibc-core/src/ics04_channel/handler.rs

@@ -92,7 +92,7 @@ impl<'a> ValidateChannel for CwIbcCoreContext<'a> {
         }
         channel_open_init_msg_validate(&channel_end, connection_end)?;
         let counter = self.channel_counter(deps.storage)?;
-        let src_channel = ChannelId::new(counter); // creating new channel_id
+        let src_channel = ChannelId::new(counter);
         let contract_address =
             self.lookup_modules(deps.storage, message.port_id.clone().as_bytes().to_vec())?;
 
@@ -101,7 +101,6 @@ impl<'a> ValidateChannel for CwIbcCoreContext<'a> {
         channel_end.state = State::Init;
         self.store_channel_end(deps.storage, &src_port, &src_channel, &channel_end)?;
 
-        // Generate event for calling on channel open init in x-call
         let sub_message =
             on_chan_open_init_submessage(&channel_end, &src_port, &src_channel, &connection_id);
 
@@ -382,7 +381,6 @@ impl<'a> ValidateChannel for CwIbcCoreContext<'a> {
         let module_contract_address =
             self.lookup_modules(deps.storage, src_port.as_bytes().to_vec())?;
 
-        // Generate event for calling on channel open try in x-call
         let sub_message =
             on_chan_open_ack_submessage(&channel_end, &src_port, &src_channel, &connection_id)?;
 
@@ -493,7 +491,7 @@ impl<'a> ValidateChannel for CwIbcCoreContext<'a> {
         let client = self.get_light_client(deps.as_ref().storage, &client_id)?;
         client.verify_channel(deps.as_ref(), verify_channel_state)?;
 
-        channel_end.set_state(State::Open); // State Change
+        channel_end.set_state(State::Open);
         self.store_channel_end(deps.storage, &dest_port, &dest_channel, &channel_end)?;
         self.store_channel_commitment(deps.storage, &dest_port, &dest_channel, &channel_end)?;
         let event = create_channel_event(
@@ -502,10 +500,9 @@ impl<'a> ValidateChannel for CwIbcCoreContext<'a> {
             dest_channel.as_str(),
             &channel_end,
         )?;
-        // Getting the module address for on channel open try call
+
         let contract_address = self.lookup_modules(deps.storage, dest_port.as_bytes().to_vec())?;
 
-        // Generate event for calling on channel open try in x-call
         let sub_message = on_chan_open_confirm_submessage(&channel_end, &dest_port, &dest_channel)?;
 
         let data =

contracts/cosmwasm-vm/cw-ibc-core/src/ics04_channel/mod.rs

@@ -14,7 +14,6 @@ pub use common::ibc::core::ics04_channel::channel::Counterparty;
 use common::ibc::core::ics04_channel::channel::Order;
 pub use common::ibc::core::ics04_channel::channel::State;
 use common::ibc::core::ics04_channel::error::{ChannelError, PacketError};
-//pub use common::ibc::core::ics04_channel::events::*;
 
 use common::ibc::events::IbcEventType;
 use cosmwasm_std::Event;

contracts/cosmwasm-vm/cw-ibc-core/src/ics04_channel/packet/receive_packet.rs

@@ -142,7 +142,6 @@ impl<'a> CwIbcCoreContext<'a> {
         cw_println!(deps, "before packet already received ");
 
         let port_id = packet.destination_port.clone();
-        // Getting the module address for on packet timeout call
         let contract_address = self.lookup_modules(deps.storage, port_id.as_bytes().to_vec())?;
 
         let src = CwEndPoint {

contracts/cosmwasm-vm/cw-ibc-core/src/ics04_channel/packet/timeout.rs

@@ -145,7 +145,6 @@ impl<'a> CwIbcCoreContext<'a> {
             next_seq_recv_verification_result,
         )?;
 
-        // Getting the module address for on packet timeout call
         let contract_address = self.lookup_modules(deps.storage, src_port.as_bytes().to_vec())?;
 
         let src = CwEndPoint {

contracts/cosmwasm-vm/cw-ibc-core/src/ics04_channel/packet/timeout_on_close.rs

@@ -172,7 +172,6 @@ impl<'a> CwIbcCoreContext<'a> {
             self.store_channel_end(deps.storage, &src_port, &src_channel, &channel_end)?;
         }
 
-        // Getting the module address for on packet timeout call
         let contract_address = self.lookup_modules(deps.storage, src_port.as_bytes().to_vec())?;
 
         let src = CwEndPoint {

contracts/cosmwasm-vm/cw-ibc-core/src/ics05_port/port.rs

@@ -64,64 +64,6 @@ impl<'a> CwIbcCoreContext<'a> {
             .save(store, port_id, &module_id)?)
     }
 
-    /// This function looks up a module ID based on a given channel message and port ID.
-    ///
-    /// Arguments:
-    ///
-    /// * `store`: `store` is a mutable reference to a trait object of type `Storage`. It is used to
-    /// interact with the storage of the smart contract.
-    /// * `msg`: A reference to a ChannelMsg enum, which represents different types of messages that can
-    /// be sent over a channel in the Cosmos SDK.
-    ///
-    /// Returns:
-    ///
-    /// a `Result` with either a `ModuleId` or a `ContractError`.
-    // pub fn lookup_module_channel(
-    //     &self,
-    //     store: &mut dyn Storage,
-    //     msg: &ChannelMsg,
-    // ) -> Result<ModuleId, ContractError> {
-    //     let port_id = match msg {
-    //         ChannelMsg::OpenInit(msg) => &msg.port_id_on_a,
-    //         ChannelMsg::OpenTry(msg) => &msg.port_id_on_b,
-    //         ChannelMsg::OpenAck(msg) => &msg.port_id_on_a,
-    //         ChannelMsg::OpenConfirm(msg) => &msg.port_id_on_b,
-    //         ChannelMsg::CloseInit(msg) => &msg.port_id_on_a,
-    //         ChannelMsg::CloseConfirm(msg) => &msg.port_id_on_b,
-    //     };
-    //     let module_id = self.lookup_module_by_port(store, port_id)?;
-    //     Ok(module_id)
-    // }
-
-    /// This function looks up a module ID based on a packet message and a storage object.
-    ///
-    /// Arguments:
-    ///
-    /// * `store`: A mutable reference to a trait object of type `Storage`. This is likely an interface
-    /// for interacting with some kind of storage system, such as a database or a key-value store.
-    /// * `msg`: `msg` is a reference to a `PacketMsg` enum, which represents different types of
-    /// messages that can be sent or received over a packet-based communication protocol. The function
-    /// uses this parameter to determine which port ID to use when looking up the corresponding module
-    /// ID.
-    ///
-    /// Returns:
-    ///
-    /// a `Result` containing either a `ModuleId` or a `ContractError`.
-    // pub fn lookup_module_packet(
-    //     &self,
-    //     store: &mut dyn Storage,
-    //     msg: &PacketMsg,
-    // ) -> Result<ModuleId, ContractError> {
-    //     let port_id = match msg {
-    //         PacketMsg::Recv(msg) => &msg.packet.port_id_on_b,
-    //         PacketMsg::Ack(msg) => &msg.packet.port_id_on_a,
-    //         PacketMsg::Timeout(msg) => &msg.packet.port_id_on_a,
-    //         PacketMsg::TimeoutOnClose(msg) => &msg.packet.port_id_on_a,
-    //     };
-    //     let module_id = self.lookup_module_by_port(store, &port_id.clone())?;
-    //     Ok(module_id)
-    // }
-
     /// This function binds a port to a given address and returns a response with relevant attributes.
     ///
     /// Arguments:

contracts/cosmwasm-vm/cw-ibc-core/tests/test_host.rs

@@ -57,7 +57,7 @@ fn test_claim_capability() {
         .store_capability(&mut deps.storage, name.clone(), address)
         .unwrap();
     let result = contract.claim_capability(&mut deps.storage, name, address_to_claim);
-    // only one address to one port
+
     assert!(result.is_err());
 }
 

contracts/cosmwasm-vm/cw-xcall-ibc-connection/tests/test_admin.rs

@@ -104,7 +104,7 @@ fn update_admin() {
     contract
         .add_admin(
             mock_deps.as_mut().storage,
-            mock_info.clone(),
+            mock_info,
             admin_one().to_string(),
         )
         .unwrap();
@@ -144,7 +144,7 @@ fn update_existing_admin() {
     contract
         .add_admin(
             mock_deps.as_mut().storage,
-            mock_info.clone(),
+            mock_info,
             admin_one().to_string(),
         )
         .unwrap();