Use if-let instead of match

CosmWasm · Aug 28, 2023 · 65cfd95 · 65cfd95
1 parent 7fe7862
commit 65cfd95
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 40 deletions.
diff --git a/libwasmvm/src/iterator.rs b/libwasmvm/src/iterator.rs
@@ -66,14 +66,13 @@ impl GoIter {
     }
 
     pub fn next(&mut self) -> BackendResult<Option<Record>> {
-        let next = match self.vtable.next {
-            Some(f) => f,
-            None => {
-                let result = Err(BackendError::unknown(
-                    "iterator vtable function 'next' not set",
-                ));
-                return (result, GasInfo::free());
-            }
+        let next = if let Some(f) = self.vtable.next {
+            f
+        } else {
+            let result = Err(BackendError::unknown(
+                "iterator vtable function 'next' not set",
+            ));
+            return (result, GasInfo::free());
         };
 
         let mut output_key = UnmanagedVector::default();
@@ -140,14 +139,13 @@ impl GoIter {
         >,
         fn_name: &str,
     ) -> BackendResult<Option<Vec<u8>>> {
-        let next = match next_fn {
-            Some(f) => f,
-            None => {
-                let result = Err(BackendError::unknown(format!(
-                    "iterator vtable function '{fn_name}' not set"
-                )));
-                return (result, GasInfo::free());
-            }
+        let next = if let Some(f) = next_fn {
+            f
+        } else {
+            let result = Err(BackendError::unknown(format!(
+                "iterator vtable function '{fn_name}' not set"
+            )));
+            return (result, GasInfo::free());
         };
 
         let mut output = UnmanagedVector::default();

diff --git a/libwasmvm/src/storage.rs b/libwasmvm/src/storage.rs
@@ -104,41 +104,38 @@ impl Storage for GoStorage {
     }
 
     fn next(&mut self, iterator_id: u32) -> BackendResult<Option<Record>> {
-        let iterator = match self.iterators.get_mut(&iterator_id) {
-            Some(i) => i,
-            None => {
-                return (
-                    Err(BackendError::iterator_does_not_exist(iterator_id)),
-                    GasInfo::free(),
-                )
-            }
+        let iterator = if let Some(i) = self.iterators.get_mut(&iterator_id) {
+            i
+        } else {
+            return (
+                Err(BackendError::iterator_does_not_exist(iterator_id)),
+                GasInfo::free(),
+            );
         };
         iterator.next()
     }
 
     fn next_key(&mut self, iterator_id: u32) -> BackendResult<Option<Vec<u8>>> {
-        let iterator = match self.iterators.get_mut(&iterator_id) {
-            Some(i) => i,
-            None => {
-                return (
-                    Err(BackendError::iterator_does_not_exist(iterator_id)),
-                    GasInfo::free(),
-                )
-            }
+        let iterator = if let Some(i) = self.iterators.get_mut(&iterator_id) {
+            i
+        } else {
+            return (
+                Err(BackendError::iterator_does_not_exist(iterator_id)),
+                GasInfo::free(),
+            );
         };
 
         iterator.next_key()
     }
 
     fn next_value(&mut self, iterator_id: u32) -> BackendResult<Option<Vec<u8>>> {
-        let iterator = match self.iterators.get_mut(&iterator_id) {
-            Some(i) => i,
-            None => {
-                return (
-                    Err(BackendError::iterator_does_not_exist(iterator_id)),
-                    GasInfo::free(),
-                )
-            }
+        let iterator = if let Some(i) = self.iterators.get_mut(&iterator_id) {
+            i
+        } else {
+            return (
+                Err(BackendError::iterator_does_not_exist(iterator_id)),
+                GasInfo::free(),
+            );
         };
 
         iterator.next_value()