about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJeroen Vannevel <jer_vannevel@outlook.com>2022-01-11 21:44:30 +0000
committerJeroen Vannevel <jer_vannevel@outlook.com>2022-01-14 01:32:11 +0000
commit08300284e7632476592a715d28653db83bdef686 (patch)
tree1b746c375eca7555cc26a1ecc3577946ebd84781
parent7bc89f25426d39220c3f533191fb0b5290fe1582 (diff)
downloadrust-08300284e7632476592a715d28653db83bdef686.tar.gz
rust-08300284e7632476592a715d28653db83bdef686.zip
removed unwrap
-rw-r--r--crates/ide_assists/src/handlers/merge_match_arms.rs23
1 files changed, 11 insertions, 12 deletions
diff --git a/crates/ide_assists/src/handlers/merge_match_arms.rs b/crates/ide_assists/src/handlers/merge_match_arms.rs
index 69d1bc0adcf..f5366bee121 100644
--- a/crates/ide_assists/src/handlers/merge_match_arms.rs
+++ b/crates/ide_assists/src/handlers/merge_match_arms.rs
@@ -101,19 +101,18 @@ fn are_same_types(
 ) -> bool {
     let arm_types = get_arm_types(&ctx, &arm);
     for other_arm_type_entry in arm_types {
-        let current_arm_type = current_arm_types.get_key_value(&other_arm_type_entry.0);
-        if current_arm_type.is_none() {
-            return false;
-        }
-
-        let unwrapped_current_arm_type = current_arm_type.unwrap().1;
-
-        if let (Some(other_arm_type), Some(current_arm_type)) =
-            (other_arm_type_entry.1, unwrapped_current_arm_type)
-        {
-            if other_arm_type.original != current_arm_type.original {
-                return false;
+        let current_arm_type_kv = current_arm_types.get_key_value(&other_arm_type_entry.0);
+        if let Some(current_arm_type) = current_arm_type_kv {
+            if let (Some(other_arm_type), Some(current_arm_type)) =
+                (other_arm_type_entry.1, current_arm_type.1)
+            {
+                if other_arm_type.original != current_arm_type.original {
+                    return false;
+                }
             }
+        } else {
+            // No corresponding field found
+            return false;
         }
     }