Fix clippy in contracts

contracts/access_control/lib.rs

@@ -1,4 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
+#![allow(clippy::let_unit_value)]
 
 pub use crate::access_control::{
     AccessControlError, ACCESS_CONTROL_PUBKEY, CHECK_ROLE_SELECTOR, HAS_ROLE_SELECTOR,
@@ -68,7 +69,7 @@ mod access_control {
         pub fn new() -> Self {
             // This call is required in order to correctly initialize the
             // `Mapping`s of our contract.
-            ink_lang::utils::initialize_contract(|contract| Self::new_init(contract))
+            ink_lang::utils::initialize_contract(Self::new_init)
         }
 
         /// Initializes the contract.
@@ -157,6 +158,12 @@ mod access_control {
         }
     }
 
+    impl Default for AccessControl {
+        fn default() -> Self {
+            Self::new()
+        }
+    }
+
     #[cfg(test)]
     mod tests {
         use ink_lang as ink;

contracts/button/lib.rs

@@ -1,4 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
+#![allow(clippy::let_unit_value)]
 
 mod errors;
 

contracts/game_token/lib.rs

@@ -1,5 +1,6 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![feature(min_specialization)]
+#![allow(clippy::let_unit_value)]
 
 pub use crate::game_token::{
     ALLOWANCE_SELECTOR, BALANCE_OF_SELECTOR, BURN_SELECTOR, MINT_SELECTOR, TRANSFER_FROM_SELECTOR,

contracts/marketplace/lib.rs

@@ -18,6 +18,7 @@
 
 #![cfg_attr(not(feature = "std"), no_std)]
 #![feature(min_specialization)]
+#![allow(clippy::let_unit_value)]
 
 use ink_lang as ink;
 
@@ -198,7 +199,7 @@ pub mod marketplace {
         /// current price is greater than that.
         #[ink(message)]
         pub fn buy(&mut self, max_price: Option<Balance>) -> Result<(), Error> {
-            if self.ticket_balance()? <= 0 {
+            if self.ticket_balance()? == 0 {
                 return Err(Error::MarketplaceEmpty);
             }
 
@@ -250,7 +251,7 @@ pub mod marketplace {
 
         fn current_price(&self) -> Balance {
             let block = self.env().block_number();
-            let elapsed = block.saturating_sub(self.current_start_block.into());
+            let elapsed = block.saturating_sub(self.current_start_block);
             self.average_price()
                 .saturating_mul(self.sale_multiplier)
                 .saturating_sub(self.per_block_reduction().saturating_mul(elapsed.into()))
@@ -312,7 +313,7 @@ pub mod marketplace {
                 Self::env().caller(),
                 role,
                 |reason| reason.into(),
-                |role| Error::MissingRole(role),
+                Error::MissingRole,
             )
         }
 

contracts/simple_dex/lib.rs

@@ -1,4 +1,5 @@
 #![cfg_attr(not(feature = "std"), no_std)]
+#![allow(clippy::let_unit_value)]
 
 use ink_lang as ink;
 
@@ -133,7 +134,7 @@ mod simple_dex {
             );
 
             match role_check {
-                Ok(_) => initialize_contract(|contract| Self::new_init(contract)),
+                Ok(_) => initialize_contract(Self::new_init),
                 Err(why) => panic!("Could not initialize the contract {:?}", why),
             }
         }
@@ -506,4 +507,10 @@ mod simple_dex {
             emitter.emit_event(event);
         }
     }
+
+    impl Default for SimpleDex {
+        fn default() -> Self {
+            SimpleDex::new()
+        }
+    }
 }

contracts/ticket_token/lib.rs

@@ -1,5 +1,6 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![feature(min_specialization)]
+#![allow(clippy::let_unit_value)]
 
 pub use crate::ticket_token::{BALANCE_OF_SELECTOR, TRANSFER_FROM_SELECTOR, TRANSFER_SELECTOR};
 

contracts/wrapped_azero/lib.rs

@@ -1,5 +1,6 @@
 #![cfg_attr(not(feature = "std"), no_std)]
 #![feature(min_specialization)]
+#![allow(clippy::let_unit_value)]
 
 pub use crate::wrapped_azero::{
     ALLOWANCE_SELECTOR, BALANCE_OF_SELECTOR, TRANSFER_FROM_SELECTOR, TRANSFER_SELECTOR,