From 91554d27529c91ea2ed845bbc80f010685235a2c Mon Sep 17 00:00:00 2001 From: Timofey <5527315+epanchee@users.noreply.github.com> Date: Wed, 12 Feb 2025 20:13:41 +0300 Subject: [PATCH] convince clippy --- contracts/pair_concentrated_duality/src/migrate.rs | 2 +- contracts/pair_concentrated_duality/src/orderbook/state.rs | 4 ++-- contracts/pair_concentrated_duality/src/orderbook/utils.rs | 4 +++- packages/astroport_test/src/modules/neutron_stargate.rs | 4 ++-- 4 files changed, 8 insertions(+), 6 deletions(-) diff --git a/contracts/pair_concentrated_duality/src/migrate.rs b/contracts/pair_concentrated_duality/src/migrate.rs index 544bd727..cc5ff856 100644 --- a/contracts/pair_concentrated_duality/src/migrate.rs +++ b/contracts/pair_concentrated_duality/src/migrate.rs @@ -53,7 +53,7 @@ pub fn migrate(deps: DepsMut, _env: Env, msg: MigrateMsg) -> StdResult StdError::generic_err(format!("This endpoint is allowed only for {CONTRACT_NAME}")) ); - Err(StdError::generic_err("Not yet implemented".to_string()).into()) + Err(StdError::generic_err("Not yet implemented".to_string())) } }?; diff --git a/contracts/pair_concentrated_duality/src/orderbook/state.rs b/contracts/pair_concentrated_duality/src/orderbook/state.rs index d4e0bf69..a712754e 100644 --- a/contracts/pair_concentrated_duality/src/orderbook/state.rs +++ b/contracts/pair_concentrated_duality/src/orderbook/state.rs @@ -222,7 +222,7 @@ impl OrderbookState { maker_coin_out, }) => Ok([taker_coin_out, maker_coin_out] .into_iter() - .filter_map(|coin| coin) + .flatten() .collect_vec()), }) }) @@ -266,7 +266,7 @@ impl OrderbookState { .into_result() .map_err(|err| StdError::generic_err(err.to_string()))? .into_result() - .map_err(|err| StdError::generic_err(err))?; + .map_err(StdError::generic_err)?; self.orders = from_json::( &response_raw, diff --git a/contracts/pair_concentrated_duality/src/orderbook/utils.rs b/contracts/pair_concentrated_duality/src/orderbook/utils.rs index 4c936d21..7c1435dc 100644 --- a/contracts/pair_concentrated_duality/src/orderbook/utils.rs +++ b/contracts/pair_concentrated_duality/src/orderbook/utils.rs @@ -213,7 +213,9 @@ fn price_to_duality_notation( ) -> Result { let prec_diff = quote_precision as i8 - base_precision as i8; let price = match prec_diff.cmp(&0) { - Ordering::Less => price / Decimal256::from_integer(10u128.pow(prec_diff.abs() as u32)), + Ordering::Less => { + price / Decimal256::from_integer(10u128.pow(prec_diff.unsigned_abs() as u32)) + } Ordering::Equal => price, Ordering::Greater => price * Decimal256::from_integer(10u128.pow(prec_diff as u32)), } diff --git a/packages/astroport_test/src/modules/neutron_stargate.rs b/packages/astroport_test/src/modules/neutron_stargate.rs index 88e8051f..aa7078f5 100644 --- a/packages/astroport_test/src/modules/neutron_stargate.rs +++ b/packages/astroport_test/src/modules/neutron_stargate.rs @@ -122,8 +122,8 @@ impl Module for NeutronStargate { self.orders .borrow_mut() .entry(sender.to_string()) - .or_insert_with(HashMap::new) - .insert(tranche_key, value.try_into()?); + .or_default() + .insert(tranche_key, value); Ok(AppResponse::default()) } MsgCancelLimitOrder::TYPE_URL => {