diff options
Diffstat (limited to 'crates/ra_ide_api/src/diagnostics.rs')
-rw-r--r-- | crates/ra_ide_api/src/diagnostics.rs | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/crates/ra_ide_api/src/diagnostics.rs b/crates/ra_ide_api/src/diagnostics.rs index 8743a3a79..1f1f5cd74 100644 --- a/crates/ra_ide_api/src/diagnostics.rs +++ b/crates/ra_ide_api/src/diagnostics.rs | |||
@@ -85,10 +85,9 @@ pub(crate) fn diagnostics(db: &RootDatabase, file_id: FileId) -> Vec<Diagnostic> | |||
85 | }) | 85 | }) |
86 | .on::<hir::diagnostics::MissingOkInTailExpr, _>(|d| { | 86 | .on::<hir::diagnostics::MissingOkInTailExpr, _>(|d| { |
87 | let node = d.ast(db); | 87 | let node = d.ast(db); |
88 | let mut builder = TextEditBuilder::default(); | ||
89 | let replacement = format!("Ok({})", node.syntax()); | 88 | let replacement = format!("Ok({})", node.syntax()); |
90 | builder.replace(node.syntax().text_range(), replacement); | 89 | let edit = TextEdit::replace(node.syntax().text_range(), replacement); |
91 | let fix = SourceChange::source_file_edit_from("wrap with ok", file_id, builder.finish()); | 90 | let fix = SourceChange::source_file_edit_from("wrap with ok", file_id, edit); |
92 | res.borrow_mut().push(Diagnostic { | 91 | res.borrow_mut().push(Diagnostic { |
93 | range: d.highlight_range(), | 92 | range: d.highlight_range(), |
94 | message: d.message(), | 93 | message: d.message(), |
@@ -152,9 +151,7 @@ fn text_edit_for_remove_unnecessary_braces_with_self_in_use_statement( | |||
152 | let start = use_tree_list_node.prev_sibling_or_token()?.text_range().start(); | 151 | let start = use_tree_list_node.prev_sibling_or_token()?.text_range().start(); |
153 | let end = use_tree_list_node.text_range().end(); | 152 | let end = use_tree_list_node.text_range().end(); |
154 | let range = TextRange::from_to(start, end); | 153 | let range = TextRange::from_to(start, end); |
155 | let mut edit_builder = TextEditBuilder::default(); | 154 | return Some(TextEdit::delete(range)); |
156 | edit_builder.delete(range); | ||
157 | return Some(edit_builder.finish()); | ||
158 | } | 155 | } |
159 | None | 156 | None |
160 | } | 157 | } |