From c2ad63539702b5a760ff9f96feb2d0e92dfe7960 Mon Sep 17 00:00:00 2001 From: toints Date: Tue, 21 May 2024 14:19:42 +0800 Subject: [PATCH] perf: removed useless code and fixed log level --- pallets/assets-bridge/src/mock.rs | 5 +---- .../precompile/transfer-to-magnet/src/lib.rs | 18 ++++++++---------- .../precompile/transfer-to-magnet/src/mock.rs | 1 - 3 files changed, 9 insertions(+), 15 deletions(-) diff --git a/pallets/assets-bridge/src/mock.rs b/pallets/assets-bridge/src/mock.rs index 092d380..4bec7f4 100644 --- a/pallets/assets-bridge/src/mock.rs +++ b/pallets/assets-bridge/src/mock.rs @@ -12,9 +12,7 @@ // along with this program. If not, see . pub use crate as assets_bridge; -pub use assets_bridge::{Config, Error, Event as AssetsBridgeEvent}; - -use sp_std::collections::btree_set::BTreeSet; +pub use assets_bridge::{Error, Event as AssetsBridgeEvent}; use frame_support::{ derive_impl, @@ -27,7 +25,6 @@ use frame_system::EnsureSigned; use sp_core::{H160, H256}; pub use sp_runtime::{ - testing::Header, traits::{BlakeTwo256, IdentityLookup}, AccountId32, BuildStorage, }; diff --git a/pallets/evm/precompile/transfer-to-magnet/src/lib.rs b/pallets/evm/precompile/transfer-to-magnet/src/lib.rs index 961d95d..7c66d67 100644 --- a/pallets/evm/precompile/transfer-to-magnet/src/lib.rs +++ b/pallets/evm/precompile/transfer-to-magnet/src/lib.rs @@ -86,7 +86,7 @@ where let target_gas = handle.gas_limit(); let context = handle.context(); - log::info!( + log::debug!( "codeAddress:{:?}, input:{:?}, targetGas:{:?}", &code_address, &input, @@ -103,9 +103,7 @@ where let token_addr = solidity::decode_arguments::
(&input[4..36])?; let amount = solidity::decode_arguments::(&input[36..68])?; - log::info!("Caller:{:?}, tokenAddr:{:?}, amount:{:?}", &caller, &token_addr, &amount); - - log::info!("who bstr data len:{:?}", &input[100..].len()); + log::debug!("Caller:{:?}, tokenAddr:{:?}, amount:{:?}", &caller, &token_addr, &amount); if handle.is_static() { log::error!("Can't be static call error"); @@ -123,12 +121,12 @@ where }); }, }; - log::info!("to who ss58:{:?}", &to_who_ss58); + log::debug!("to who ss58:{:?}", &to_who_ss58); let to_who_id32 = AccountId32::from_ss58check(&to_who_ss58).map_err(|_| PrecompileFailure::Error { exit_status: ExitError::Other("AccountId32 from ss58check(string) failed".into()), })?; - log::info!("to_who_ss58:{:?}, to_who_id32:{:?}", &to_who_ss58, &to_who_id32); + log::debug!("to_who_ss58:{:?}, to_who_id32:{:?}", &to_who_ss58, &to_who_id32); let to_who: ::AccountId = to_who_id32.clone().into(); @@ -153,7 +151,7 @@ where let amount_saturated: T::Balance = amount.into(); - log::info!( + log::debug!( "Preparing to mint: AssetId: {:?}, Beneficiary: {:?}, Amount: {:?}", &asset_id, &to_who_ss58, @@ -238,9 +236,9 @@ where } let length_bytes = &input[offset..offset + 32]; - log::info!("length_bytes:{:?}", &length_bytes); + log::debug!("length_bytes:{:?}", &length_bytes); let length = u32::from_be_bytes(length_bytes[28..32].try_into().unwrap()) as usize; - log::info!("ss58 string len:{:?}", length); + log::debug!("ss58 string len:{:?}", length); if input.len() < offset + 32 + length { return Err("Input too short to contain string data"); @@ -249,7 +247,7 @@ where let string_data_start = offset + 32; let string_data_end = string_data_start + length; let string_data = &input[string_data_start..string_data_end]; - log::info!("ss58 string data:{:?}", &string_data); + log::debug!("ss58 string data:{:?}", &string_data); let mut result_string = from_utf8(string_data) .map_err(|_| "String data is not valid UTF-8")? diff --git a/pallets/evm/precompile/transfer-to-magnet/src/mock.rs b/pallets/evm/precompile/transfer-to-magnet/src/mock.rs index d1e16eb..b1a9f47 100644 --- a/pallets/evm/precompile/transfer-to-magnet/src/mock.rs +++ b/pallets/evm/precompile/transfer-to-magnet/src/mock.rs @@ -35,7 +35,6 @@ use sp_runtime::{ traits::{BlakeTwo256, IdentityLookup}, BuildStorage, }; -use sp_std::collections::{btree_map::BTreeMap, btree_set::BTreeSet}; use std::str::FromStr; use xcm::latest::prelude::BodyId;