address linting issues from clippy

public-awesome · Apr 26, 2024 · 5e50c1c · 5e50c1c
1 parent eb59c69
commit 5e50c1c
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/contracts/marketplace/src/helpers.rs b/contracts/marketplace/src/helpers.rs
@@ -209,7 +209,7 @@ fn sell_name(
         deps.as_ref(),
         ask.clone(),
         bid.amount,
-        bid.bidder.clone(),
+        bid.bidder,
         &mut response,
     )?;
 

diff --git a/contracts/name-minter/src/contract.rs b/contracts/name-minter/src/contract.rs
@@ -178,7 +178,7 @@ pub fn execute_mint_and_list(
         .unwrap_or(None);
 
     let price = validate_payment(name.len(), &info, params.base_price.u128(), discount)?;
-    if price.clone().is_some() {
+    if price.is_some() {
         charge_fees(
             &mut res,
             params.fair_burn_percent,
@@ -216,7 +216,7 @@ pub fn execute_mint_and_list(
         .add_attribute("owner", sender)
         .add_attribute(
             "price",
-            price.unwrap_or(coin(0, NATIVE_DENOM)).amount.to_string(),
+            price.unwrap_or_else(|| coin(0, NATIVE_DENOM)).amount.to_string(),
         );
     Ok(res
         .add_event(event)

diff --git a/contracts/sg721-name/src/contract.rs b/contracts/sg721-name/src/contract.rs
@@ -615,7 +615,7 @@ fn validate_address(deps: Deps, sender: &Addr, addr: Addr) -> Result<Addr, Contr
         let collection_info: CollectionInfoResponse = deps
             .querier
             .query_wasm_smart(&addr, &sg721_base::msg::QueryMsg::CollectionInfo {})?;
-        if collection_info.creator == sender.to_string() {
+        if collection_info.creator == *sender.to_string() {
             return Ok(addr);
         }
     }

diff --git a/contracts/whitelist-updatable-flatrate/src/integration_tests.rs b/contracts/whitelist-updatable-flatrate/src/integration_tests.rs
@@ -421,7 +421,7 @@ mod tests {
 
         let msg = InstantiateMsg {
             per_address_limit: PER_ADDRESS_LIMIT,
-            addresses: addrs.clone(),
+            addresses: addrs,
             mint_discount_amount: None,
             admin_list: Some(vec![CREATOR.to_string(), TEMP_ADMIN.to_string()]),
         };
@@ -458,7 +458,7 @@ mod tests {
         let msg = sg_name_minter::SgNameMinterExecuteMsg::AddWhitelist {
             address: wl_addr.to_string(),
         };
-        let res = app.execute_contract(Addr::unchecked(CREATOR), minter_addr.clone(), &msg, &[]);
+        let res = app.execute_contract(Addr::unchecked(CREATOR), minter_addr, &msg, &[]);
         assert!(res.is_ok());
 
         let res: bool = app