Unnamed repository; edit this file 'description' to name the repository.
internal: Collapse lift is_infer_qualifier into `Qualified` variant
Lukas Wirth 2022-06-17
parent 2f2ea77 · commit d6f161f
-rw-r--r--crates/ide-completion/src/completions/attribute.rs1
-rw-r--r--crates/ide-completion/src/completions/attribute/derive.rs1
-rw-r--r--crates/ide-completion/src/completions/expr.rs16
-rw-r--r--crates/ide-completion/src/completions/item_list.rs2
-rw-r--r--crates/ide-completion/src/completions/pattern.rs1
-rw-r--r--crates/ide-completion/src/completions/type.rs16
-rw-r--r--crates/ide-completion/src/completions/use_.rs9
-rw-r--r--crates/ide-completion/src/completions/vis.rs2
-rw-r--r--crates/ide-completion/src/context.rs8
-rw-r--r--crates/ide-completion/src/context/analysis.rs25
10 files changed, 41 insertions, 40 deletions
diff --git a/crates/ide-completion/src/completions/attribute.rs b/crates/ide-completion/src/completions/attribute.rs
index 56e51d32e2..096e24eab3 100644
--- a/crates/ide-completion/src/completions/attribute.rs
+++ b/crates/ide-completion/src/completions/attribute.rs
@@ -112,6 +112,7 @@ pub(crate) fn complete_attribute(acc: &mut Completions, ctx: &CompletionContext)
});
acc.add_nameref_keywords_with_colon(ctx);
}
+ Qualified::Infer => {}
}
let attributes = annotated_item_kind.and_then(|kind| {
diff --git a/crates/ide-completion/src/completions/attribute/derive.rs b/crates/ide-completion/src/completions/attribute/derive.rs
index 3aaaecd474..0f78fc988d 100644
--- a/crates/ide-completion/src/completions/attribute/derive.rs
+++ b/crates/ide-completion/src/completions/attribute/derive.rs
@@ -101,6 +101,7 @@ pub(crate) fn complete_derive(acc: &mut Completions, ctx: &CompletionContext) {
});
acc.add_nameref_keywords_with_colon(ctx);
}
+ Qualified::Infer => {}
}
}
diff --git a/crates/ide-completion/src/completions/expr.rs b/crates/ide-completion/src/completions/expr.rs
index f0a8529c78..41e1dc9f91 100644
--- a/crates/ide-completion/src/completions/expr.rs
+++ b/crates/ide-completion/src/completions/expr.rs
@@ -61,15 +61,13 @@ pub(crate) fn complete_expr_path(acc: &mut Completions, ctx: &CompletionContext)
};
match qualified {
- Qualified::With(PathQualifierCtx { is_infer_qualifier, resolution, .. }) => {
- if *is_infer_qualifier {
- ctx.traits_in_scope()
- .0
- .into_iter()
- .flat_map(|it| hir::Trait::from(it).items(ctx.sema.db))
- .for_each(|item| add_assoc_item(acc, ctx, item));
- return;
- }
+ Qualified::Infer => ctx
+ .traits_in_scope()
+ .0
+ .into_iter()
+ .flat_map(|it| hir::Trait::from(it).items(ctx.sema.db))
+ .for_each(|item| add_assoc_item(acc, ctx, item)),
+ Qualified::With(PathQualifierCtx { resolution, .. }) => {
let resolution = match resolution {
Some(it) => it,
None => return,
diff --git a/crates/ide-completion/src/completions/item_list.rs b/crates/ide-completion/src/completions/item_list.rs
index e3f3b7f528..9c8fbb84f5 100644
--- a/crates/ide-completion/src/completions/item_list.rs
+++ b/crates/ide-completion/src/completions/item_list.rs
@@ -66,7 +66,7 @@ pub(crate) fn complete_item_list(acc: &mut Completions, ctx: &CompletionContext)
});
acc.add_nameref_keywords_with_colon(ctx);
}
- Qualified::No => {}
+ Qualified::Infer | Qualified::No => {}
}
}
diff --git a/crates/ide-completion/src/completions/pattern.rs b/crates/ide-completion/src/completions/pattern.rs
index b7710ae70d..8dedcc35ec 100644
--- a/crates/ide-completion/src/completions/pattern.rs
+++ b/crates/ide-completion/src/completions/pattern.rs
@@ -208,5 +208,6 @@ fn pattern_path_completion(
acc.add_nameref_keywords_with_colon(ctx);
}
+ Qualified::Infer => {}
}
}
diff --git a/crates/ide-completion/src/completions/type.rs b/crates/ide-completion/src/completions/type.rs
index f2418d76cc..d2dab9f4ed 100644
--- a/crates/ide-completion/src/completions/type.rs
+++ b/crates/ide-completion/src/completions/type.rs
@@ -55,15 +55,13 @@ pub(crate) fn complete_type_path(acc: &mut Completions, ctx: &CompletionContext)
};
match qualified {
- Qualified::With(PathQualifierCtx { is_infer_qualifier, resolution, .. }) => {
- if *is_infer_qualifier {
- ctx.traits_in_scope()
- .0
- .into_iter()
- .flat_map(|it| hir::Trait::from(it).items(ctx.sema.db))
- .for_each(|item| add_assoc_item(acc, item));
- return;
- }
+ Qualified::Infer => ctx
+ .traits_in_scope()
+ .0
+ .into_iter()
+ .flat_map(|it| hir::Trait::from(it).items(ctx.sema.db))
+ .for_each(|item| add_assoc_item(acc, item)),
+ Qualified::With(PathQualifierCtx { resolution, .. }) => {
let resolution = match resolution {
Some(it) => it,
None => return,
diff --git a/crates/ide-completion/src/completions/use_.rs b/crates/ide-completion/src/completions/use_.rs
index 3b6ce8fc1a..33b50f22f5 100644
--- a/crates/ide-completion/src/completions/use_.rs
+++ b/crates/ide-completion/src/completions/use_.rs
@@ -24,13 +24,7 @@ pub(crate) fn complete_use_tree(acc: &mut Completions, ctx: &CompletionContext)
};
match qualified {
- Qualified::With(PathQualifierCtx {
- path,
- resolution,
- is_super_chain,
- use_tree_parent,
- ..
- }) => {
+ Qualified::With(PathQualifierCtx { path, resolution, is_super_chain, use_tree_parent }) => {
if *is_super_chain {
acc.add_keyword(ctx, "super::");
}
@@ -136,5 +130,6 @@ pub(crate) fn complete_use_tree(acc: &mut Completions, ctx: &CompletionContext)
});
acc.add_nameref_keywords_with_colon(ctx);
}
+ Qualified::Infer => {}
}
}
diff --git a/crates/ide-completion/src/completions/vis.rs b/crates/ide-completion/src/completions/vis.rs
index b971808626..d27fc8f446 100644
--- a/crates/ide-completion/src/completions/vis.rs
+++ b/crates/ide-completion/src/completions/vis.rs
@@ -37,7 +37,7 @@ pub(crate) fn complete_vis_path(acc: &mut Completions, ctx: &CompletionContext)
acc.add_keyword(ctx, "super::");
}
}
- Qualified::Absolute => {}
+ Qualified::Absolute | Qualified::Infer => {}
Qualified::No => {
if !has_in_token {
cov_mark::hit!(kw_completion_in);
diff --git a/crates/ide-completion/src/context.rs b/crates/ide-completion/src/context.rs
index 781dcfd403..e2bf4284f5 100644
--- a/crates/ide-completion/src/context.rs
+++ b/crates/ide-completion/src/context.rs
@@ -150,6 +150,8 @@ pub(super) enum ItemListKind {
pub(super) enum Qualified {
No,
With(PathQualifierCtx),
+ /// <_>::
+ Infer,
/// Whether the path is an absolute path
Absolute,
}
@@ -163,8 +165,6 @@ pub(crate) struct PathQualifierCtx {
pub(crate) is_super_chain: bool,
/// Whether the qualifier comes from a use tree parent or not
pub(crate) use_tree_parent: bool,
- /// <_>
- pub(crate) is_infer_qualifier: bool,
}
/// The state of the pattern we are completing.
@@ -320,13 +320,16 @@ pub(crate) struct CompletionContext<'a> {
pub(super) expected_type: Option<Type>,
/// The parent function of the cursor position if it exists.
+ // FIXME: This probably doesn't belong here
pub(super) function_def: Option<ast::Fn>,
/// The parent impl of the cursor position if it exists.
+ // FIXME: This probably doesn't belong here
pub(super) impl_def: Option<ast::Impl>,
/// Are we completing inside a let statement with a missing semicolon?
// FIXME: This should be part of PathKind::Expr
pub(super) incomplete_let: bool,
+ // FIXME: This shouldn't exist
pub(super) previous_token: Option<SyntaxToken>,
pub(super) ident_ctx: IdentContext,
@@ -354,6 +357,7 @@ impl<'a> CompletionContext<'a> {
}
}
+ // FIXME: This shouldn't exist
pub(crate) fn previous_token_is(&self, kind: SyntaxKind) -> bool {
self.previous_token.as_ref().map_or(false, |tok| tok.kind() == kind)
}
diff --git a/crates/ide-completion/src/context/analysis.rs b/crates/ide-completion/src/context/analysis.rs
index 1989c3365f..57d6682b4b 100644
--- a/crates/ide-completion/src/context/analysis.rs
+++ b/crates/ide-completion/src/context/analysis.rs
@@ -861,10 +861,6 @@ impl<'a> CompletionContext<'a> {
.and_then(|it| find_node_in_file(original_file, &it))
.map(|it| it.parent_path());
if let Some(path) = path {
- let res = sema.resolve_path(&path);
- let is_super_chain = iter::successors(Some(path.clone()), |p| p.qualifier())
- .all(|p| p.segment().and_then(|s| s.super_token()).is_some());
-
// `<_>::$0`
let is_infer_qualifier = path.qualifier().is_none()
&& matches!(
@@ -875,13 +871,20 @@ impl<'a> CompletionContext<'a> {
})
);
- path_ctx.qualified = Qualified::With(PathQualifierCtx {
- path,
- resolution: res,
- is_super_chain,
- use_tree_parent,
- is_infer_qualifier,
- })
+ path_ctx.qualified = if is_infer_qualifier {
+ Qualified::Infer
+ } else {
+ let res = sema.resolve_path(&path);
+ let is_super_chain =
+ iter::successors(Some(path.clone()), |p| p.qualifier())
+ .all(|p| p.segment().and_then(|s| s.super_token()).is_some());
+ Qualified::With(PathQualifierCtx {
+ path,
+ resolution: res,
+ is_super_chain,
+ use_tree_parent,
+ })
+ }
};
}
} else if let Some(segment) = path.segment() {