fix: Inline value match in SparseTrie::find_leaf to remove redundant wrapper

crates/trie/sparse/src/trie.rs

@@ -969,24 +969,6 @@ impl SparseTrieInterface for SerialSparseTrie {
         full_path: &Nibbles,
         expected_value: Option<&Vec<u8>>,
     ) -> Result<LeafLookup, LeafLookupError> {
-        // Helper function to check if a value matches the expected value
-        fn check_value_match(
-            actual_value: &Vec<u8>,
-            expected_value: Option<&Vec<u8>>,
-            path: &Nibbles,
-        ) -> Result<(), LeafLookupError> {
-            if let Some(expected) = expected_value &&
-                actual_value != expected
-            {
-                return Err(LeafLookupError::ValueMismatch {
-                    path: *path,
-                    expected: Some(expected.clone()),
-                    actual: actual_value.clone(),
-                });
-            }
-            Ok(())
-        }
-
         let mut current = Nibbles::default(); // Start at the root
 
         // Inclusion proof
@@ -996,7 +978,13 @@ impl SparseTrieInterface for SerialSparseTrie {
         // be in the `values` map.
         if let Some(actual_value) = self.values.get(full_path) {
             // We found the leaf, check if the value matches (if expected value was provided)
-            check_value_match(actual_value, expected_value, full_path)?;
+            if expected_value.is_some_and(|expected| actual_value != expected) {
+                return Err(LeafLookupError::ValueMismatch {
+                    path: *full_path,
+                    expected: expected_value.cloned(),
+                    actual: actual_value.clone(),
+                });
+            }
             return Ok(LeafLookup::Exists);
         }
 
@@ -1023,7 +1011,13 @@ impl SparseTrieInterface for SerialSparseTrie {
                     if &current == full_path {
                         // This should have been handled by our initial values map check
                         if let Some(value) = self.values.get(full_path) {
-                            check_value_match(value, expected_value, full_path)?;
+                            if expected_value.is_some_and(|expected| value != expected) {
+                                return Err(LeafLookupError::ValueMismatch {
+                                    path: *full_path,
+                                    expected: expected_value.cloned(),
+                                    actual: value.clone(),
+                                });
+                            }
                             return Ok(LeafLookup::Exists);
                         }
                     }