Skip to content

Commit 035a373

Browse files
committed
removed double matching
1 parent 771c87f commit 035a373

File tree

1 file changed

+4
-17
lines changed

1 file changed

+4
-17
lines changed

crates/ide_assists/src/handlers/extract_variable.rs

Lines changed: 4 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ pub(crate) fn extract_variable(acc: &mut Assists, ctx: &AssistContext) -> Option
5252
}
5353
}
5454

55-
let ref_kind = match get_receiver_type(&ctx, &to_extract) {
56-
Some(receiver_type) if receiver_type.is_mutable_reference() => RefKind::MutRef,
57-
Some(receiver_type) if receiver_type.is_reference() => RefKind::Ref,
58-
_ => RefKind::None,
55+
let reference_modifier = match get_receiver_type(&ctx, &to_extract) {
56+
Some(receiver_type) if receiver_type.is_mutable_reference() => "&mut ",
57+
Some(receiver_type) if receiver_type.is_reference() => "&",
58+
_ => "",
5959
};
6060

6161
let anchor = Anchor::from(&to_extract)?;
@@ -83,12 +83,6 @@ pub(crate) fn extract_variable(acc: &mut Assists, ctx: &AssistContext) -> Option
8383
None => to_extract.syntax().text_range(),
8484
};
8585

86-
let reference_modifier = match ref_kind {
87-
RefKind::MutRef => "&mut ",
88-
RefKind::Ref => "&",
89-
RefKind::None => "",
90-
};
91-
9286
match anchor {
9387
Anchor::Before(_) | Anchor::Replace(_) => {
9488
format_to!(buf, "let {} = {}", var_name, reference_modifier)
@@ -176,13 +170,6 @@ fn get_receiver(expression: ast::Expr) -> Option<ast::Expr> {
176170
}
177171
}
178172

179-
#[derive(Debug)]
180-
enum RefKind {
181-
Ref,
182-
MutRef,
183-
None,
184-
}
185-
186173
#[derive(Debug)]
187174
enum Anchor {
188175
Before(SyntaxNode),

0 commit comments

Comments
 (0)