From 687bec117c0a59dc58daf5c9954c906eeb60de21 Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Fri, 13 Mar 2020 17:58:49 +0100 Subject: Don't use generic DB where a concrete one will do --- crates/ra_ide/src/completion/presentation.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'crates/ra_ide/src') diff --git a/crates/ra_ide/src/completion/presentation.rs b/crates/ra_ide/src/completion/presentation.rs index 910844244..253848602 100644 --- a/crates/ra_ide/src/completion/presentation.rs +++ b/crates/ra_ide/src/completion/presentation.rs @@ -1,6 +1,6 @@ //! This modules takes care of rendering various definitions as completion items. -use hir::{db::HirDatabase, Docs, HasAttrs, HasSource, HirDisplay, ScopeDef, StructKind, Type}; +use hir::{Docs, HasAttrs, HasSource, HirDisplay, ScopeDef, StructKind, Type}; use join_to_string::join; use ra_syntax::ast::NameOwner; use test_utils::tested_by; @@ -9,7 +9,10 @@ use crate::completion::{ CompletionContext, CompletionItem, CompletionItemKind, CompletionKind, Completions, }; -use crate::display::{const_label, macro_label, type_label, FunctionSignature}; +use crate::{ + display::{const_label, macro_label, type_label, FunctionSignature}, + RootDatabase, +}; impl Completions { pub(crate) fn add_field( @@ -300,7 +303,7 @@ impl Completions { } } -fn is_deprecated(node: impl HasAttrs, db: &impl HirDatabase) -> bool { +fn is_deprecated(node: impl HasAttrs, db: &RootDatabase) -> bool { node.attrs(db).by_key("deprecated").exists() } -- cgit v1.2.3 From a6638af4a60688c970ac9bafdf3c2d79aa1bc13c Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Fri, 13 Mar 2020 18:00:23 +0100 Subject: Simplify tests --- crates/ra_ide/src/typing.rs | 40 ++++++++++++++++++++-------------------- 1 file changed, 20 insertions(+), 20 deletions(-) (limited to 'crates/ra_ide/src') diff --git a/crates/ra_ide/src/typing.rs b/crates/ra_ide/src/typing.rs index 53c65f8bc..cb2cd2479 100644 --- a/crates/ra_ide/src/typing.rs +++ b/crates/ra_ide/src/typing.rs @@ -213,14 +213,14 @@ fn foo() { type_char( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() <|> } ", r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() . } ", @@ -228,8 +228,8 @@ fn foo() { type_char_noop( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() <|> } ", @@ -241,14 +241,14 @@ fn foo() { type_char( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() <|>; } ", r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() .; } ", @@ -256,8 +256,8 @@ fn foo() { type_char_noop( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() <|>; } ", @@ -269,15 +269,15 @@ fn foo() { type_char( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() .first() <|> } ", r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() .first() . } @@ -286,8 +286,8 @@ fn foo() { type_char_noop( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { - self.child_impl(db, name) + fn main() { + xs.foo() .first() <|> } @@ -334,7 +334,7 @@ fn foo() { type_char_noop( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { + fn main() { <|> } ", @@ -342,7 +342,7 @@ fn foo() { type_char_noop( '.', r" - pub fn child(&self, db: &impl HirDatabase, name: &Name) -> Cancelable> { + fn main() { <|> } ", -- cgit v1.2.3