diff options
| author | Jeroen Vannevel <jer_vannevel@outlook.com> | 2022-01-11 21:44:30 +0000 |
|---|---|---|
| committer | Jeroen Vannevel <jer_vannevel@outlook.com> | 2022-01-14 01:32:11 +0000 |
| commit | 08300284e7632476592a715d28653db83bdef686 (patch) | |
| tree | 1b746c375eca7555cc26a1ecc3577946ebd84781 | |
| parent | 7bc89f25426d39220c3f533191fb0b5290fe1582 (diff) | |
| download | rust-08300284e7632476592a715d28653db83bdef686.tar.gz rust-08300284e7632476592a715d28653db83bdef686.zip | |
removed unwrap
| -rw-r--r-- | crates/ide_assists/src/handlers/merge_match_arms.rs | 23 |
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; } } |
