about summary refs log tree commit diff
diff options
context:
space:
mode:
authorLaurențiu Nicola <lnicola@dend.ro>2021-10-16 13:39:55 +0300
committerLaurențiu Nicola <lnicola@dend.ro>2021-10-16 13:39:55 +0300
commit2bf5f14666084632e25d1af86cd0fea6792d60e0 (patch)
tree7d45a866935542d237f62b16b76d52200095af24
parent2efcff7f75a9a88d50600d4208db6b33be2705e3 (diff)
downloadrust-2bf5f14666084632e25d1af86cd0fea6792d60e0.tar.gz
rust-2bf5f14666084632e25d1af86cd0fea6792d60e0.zip
Use trimmed selection range
-rw-r--r--crates/ide_assists/src/handlers/extract_module.rs18
1 files changed, 9 insertions, 9 deletions
diff --git a/crates/ide_assists/src/handlers/extract_module.rs b/crates/ide_assists/src/handlers/extract_module.rs
index e466b3990e6..dbff8f3d3e8 100644
--- a/crates/ide_assists/src/handlers/extract_module.rs
+++ b/crates/ide_assists/src/handlers/extract_module.rs
@@ -51,7 +51,7 @@ use super::remove_unused_param::range_to_remove;
 // }
 // ```
 pub(crate) fn extract_module(acc: &mut Assists, ctx: &AssistContext) -> Option<()> {
-    if ctx.frange.range.is_empty() {
+    if ctx.has_empty_selection() {
         return None;
     }
 
@@ -66,7 +66,7 @@ pub(crate) fn extract_module(acc: &mut Assists, ctx: &AssistContext) -> Option<(
         curr_parent_module = ast::Module::cast(mod_syn_opt);
     }
 
-    let mut module = extract_target(&node, ctx.frange.range)?;
+    let mut module = extract_target(&node, ctx.selection_trimmed())?;
     if module.body_items.len() == 0 {
         return None;
     }
@@ -119,7 +119,7 @@ pub(crate) fn extract_module(acc: &mut Assists, ctx: &AssistContext) -> Option<(
 
             let mut usages_to_be_updated_for_curr_file = vec![];
             for usages_to_be_updated_for_file in usages_to_be_processed {
-                if usages_to_be_updated_for_file.0 == ctx.frange.file_id {
+                if usages_to_be_updated_for_file.0 == ctx.file_id() {
                     usages_to_be_updated_for_curr_file = usages_to_be_updated_for_file.1;
                     continue;
                 }
@@ -129,7 +129,7 @@ pub(crate) fn extract_module(acc: &mut Assists, ctx: &AssistContext) -> Option<(
                 }
             }
 
-            builder.edit_file(ctx.frange.file_id);
+            builder.edit_file(ctx.file_id());
             for usage_to_be_processed in usages_to_be_updated_for_curr_file {
                 builder.replace(usage_to_be_processed.0, usage_to_be_processed.1)
             }
@@ -426,11 +426,11 @@ impl Module {
         ctx: &AssistContext,
     ) -> Option<TextRange> {
         //We only need to find in the current file
-        let selection_range = ctx.frange.range;
-        let search_scope = SearchScope::single_file(ctx.frange.file_id);
+        let selection_range = ctx.selection_trimmed();
+        let curr_file_id = ctx.file_id();
+        let search_scope = SearchScope::single_file(curr_file_id);
         let usage_res = def.usages(&ctx.sema).in_scope(search_scope).all();
-        let curr_file_id = ctx.frange.file_id;
-        let file = ctx.sema.parse(ctx.frange.file_id);
+        let file = ctx.sema.parse(curr_file_id);
 
         let mut exists_inside_sel = false;
         let mut exists_outside_sel = false;
@@ -470,7 +470,7 @@ impl Module {
         let use_stmt_opt: Option<ast::Use> = usage_res.into_iter().find_map(|x| {
             let file_id = x.0;
             let mut use_opt: Option<ast::Use> = None;
-            if file_id == ctx.frange.file_id {
+            if file_id == curr_file_id {
                 (&x.1).into_iter().for_each(|x| {
                     let node_opt: Option<ast::Use> = find_node_at_range(file.syntax(), x.range);
                     if let Some(node) = node_opt {