diff options
| author | Ville Penttinen <[email protected]> | 2019-03-24 20:53:41 +0000 |
|---|---|---|
| committer | Ville Penttinen <[email protected]> | 2019-03-24 20:53:41 +0000 |
| commit | 22e1c7a112832a18509d400841b3d162228372bf (patch) | |
| tree | 0d578585dc4d706a910e248d8525e5537b952fb5 /crates/ra_ide_api/src/diagnostics.rs | |
| parent | 449eea11617e79c90b5d8de0959ef2bbe2a5d730 (diff) | |
Add convenience functions to SourceChange for creating single edits
Diffstat (limited to 'crates/ra_ide_api/src/diagnostics.rs')
| -rw-r--r-- | crates/ra_ide_api/src/diagnostics.rs | 27 |
1 files changed, 9 insertions, 18 deletions
diff --git a/crates/ra_ide_api/src/diagnostics.rs b/crates/ra_ide_api/src/diagnostics.rs index b9dc424c6..b0f23773e 100644 --- a/crates/ra_ide_api/src/diagnostics.rs +++ b/crates/ra_ide_api/src/diagnostics.rs | |||
| @@ -71,12 +71,10 @@ fn check_unnecessary_braces_in_use_statement( | |||
| 71 | range, | 71 | range, |
| 72 | message: format!("Unnecessary braces in use statement"), | 72 | message: format!("Unnecessary braces in use statement"), |
| 73 | severity: Severity::WeakWarning, | 73 | severity: Severity::WeakWarning, |
| 74 | fix: Some(SourceChange { | 74 | fix: Some(SourceChange::source_edit( |
| 75 | label: "Remove unnecessary braces".to_string(), | 75 | "Remove unnecessary braces", |
| 76 | source_file_edits: vec![SourceFileEdit { file_id, edit }], | 76 | SourceFileEdit { file_id, edit }, |
| 77 | file_system_edits: Vec::new(), | 77 | )), |
| 78 | cursor_position: None, | ||
| 79 | }), | ||
| 80 | }); | 78 | }); |
| 81 | } | 79 | } |
| 82 | 80 | ||
| @@ -119,12 +117,10 @@ fn check_struct_shorthand_initialization( | |||
| 119 | range: named_field.syntax().range(), | 117 | range: named_field.syntax().range(), |
| 120 | message: format!("Shorthand struct initialization"), | 118 | message: format!("Shorthand struct initialization"), |
| 121 | severity: Severity::WeakWarning, | 119 | severity: Severity::WeakWarning, |
| 122 | fix: Some(SourceChange { | 120 | fix: Some(SourceChange::source_edit( |
| 123 | label: "use struct shorthand initialization".to_string(), | 121 | "use struct shorthand initialization", |
| 124 | source_file_edits: vec![SourceFileEdit { file_id, edit }], | 122 | SourceFileEdit { file_id, edit }, |
| 125 | file_system_edits: Vec::new(), | 123 | )), |
| 126 | cursor_position: None, | ||
| 127 | }), | ||
| 128 | }); | 124 | }); |
| 129 | } | 125 | } |
| 130 | } | 126 | } |
| @@ -144,12 +140,7 @@ fn check_module( | |||
| 144 | Problem::UnresolvedModule { candidate } => { | 140 | Problem::UnresolvedModule { candidate } => { |
| 145 | let create_file = | 141 | let create_file = |
| 146 | FileSystemEdit::CreateFile { source_root, path: candidate.clone() }; | 142 | FileSystemEdit::CreateFile { source_root, path: candidate.clone() }; |
| 147 | let fix = SourceChange { | 143 | let fix = SourceChange::system_edit("create module", create_file); |
| 148 | label: "create module".to_string(), | ||
| 149 | source_file_edits: Vec::new(), | ||
| 150 | file_system_edits: vec![create_file], | ||
| 151 | cursor_position: None, | ||
| 152 | }; | ||
| 153 | Diagnostic { | 144 | Diagnostic { |
| 154 | range: name_node.range(), | 145 | range: name_node.range(), |
| 155 | message: "unresolved module".to_string(), | 146 | message: "unresolved module".to_string(), |