chore: fix clippy lints

RGB-WG · Aug 9, 2024 · a90e556 · a90e556
1 parent 1ec32eb
commit a90e556
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 32 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -99,4 +99,4 @@ features = ["all"]
 [patch.crates-io]
 bp-consensus = { git = "https://github.com/BP-WG/bp-core", branch = "master" }
 bp-invoice = { git = "https://github.com/BP-WG/bp-std.git", branch = "master" }
-rgb-core = { git = "https://github.com/RGB-WG/rgb-core", branch = "contract-state2" }
+rgb-core = { git = "https://github.com/RGB-WG/rgb-core", branch = "master" }
diff --git a/src/containers/consignment.rs b/src/containers/consignment.rs
@@ -536,7 +536,7 @@ Check-SHA256: aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
     #[test]
     fn error_transfer_strs() {
         let s = include_str!("../../asset/armored_transfer.default");
-        assert!(matches!(Transfer::from_str(s), Ok(_)));
+        assert!(Transfer::from_str(s).is_ok());
 
         // Wrong Id
         assert!(

diff --git a/src/persistence/memory.rs b/src/persistence/memory.rs
@@ -1000,36 +1000,34 @@ impl ContractStateEvolve for MemContract<MemContractState> {
     }
 
     fn evolve_state(&mut self, op: OrdOpRef) -> Result<(), confinement::Error> {
-        (move || -> Result<(), SerializeError> {
-            fn writer(me: &mut MemContract<MemContractState>) -> MemContractWriter {
-                MemContractWriter {
-                    writer: Box::new(
-                        |witness_id: XWitnessId, ord: WitnessOrd| -> Result<(), SerializeError> {
-                            // NB: We do not check the existence of the witness since we have a
-                            // newer version anyway and even if it is
-                            // known we have to replace it
-                            me.filter.insert(witness_id, ord);
-                            Ok(())
-                        },
-                    ),
-                    contract: &mut me.unfiltered,
-                }
+        fn writer(me: &mut MemContract<MemContractState>) -> MemContractWriter {
+            MemContractWriter {
+                writer: Box::new(
+                    |witness_id: XWitnessId, ord: WitnessOrd| -> Result<(), SerializeError> {
+                        // NB: We do not check the existence of the witness since we have a
+                        // newer version anyway and even if it is
+                        // known we have to replace it
+                        me.filter.insert(witness_id, ord);
+                        Ok(())
+                    },
+                ),
+                contract: &mut me.unfiltered,
             }
-            match op {
-                OrdOpRef::Genesis(genesis) => {
-                    let mut writer = writer(self);
-                    writer.add_genesis(genesis)
-                }
-                OrdOpRef::Transition(transition, witness_id, ord) => {
-                    let mut writer = writer(self);
-                    writer.add_transition(transition, witness_id, ord)
-                }
-                OrdOpRef::Extension(extension, witness_id, ord) => {
-                    let mut writer = writer(self);
-                    writer.add_extension(extension, witness_id, ord)
-                }
+        }
+        match op {
+            OrdOpRef::Genesis(genesis) => {
+                let mut writer = writer(self);
+                writer.add_genesis(genesis)
             }
-        })()
+            OrdOpRef::Transition(transition, witness_id, ord) => {
+                let mut writer = writer(self);
+                writer.add_transition(transition, witness_id, ord)
+            }
+            OrdOpRef::Extension(extension, witness_id, ord) => {
+                let mut writer = writer(self);
+                writer.add_extension(extension, witness_id, ord)
+            }
+        }
         .map_err(|err| match err {
             SerializeError::Io(_) => {
                 unreachable!("I/O errors are not possible for memory structures")

diff --git a/src/persistence/stock.rs b/src/persistence/stock.rs
@@ -549,7 +549,7 @@ impl<S: StashProvider, H: StateProvider, P: IndexProvider> Stock<S, H, P> {
             .map_err(StockError::from)
     }
 
-    #[allow(clippy::multiple_bound_locations)]
+    #[allow(clippy::multiple_bound_locations, clippy::type_complexity)]
     pub fn contract_iface_class<C: IfaceClass>(
         &self,
         contract_id: ContractId,