synchronizing changes

This commit is contained in:
Dmitry 2020-08-15 01:43:34 +07:00
parent 178c3e135a
commit 73315c9168
6 changed files with 28 additions and 14 deletions

View file

@ -1,4 +1,4 @@
//! FIXME: write short doc here //! Provides set of implementation for hir's objects that allows get back location in file.
use either::Either; use either::Either;
use hir_def::{ use hir_def::{

View file

@ -18,6 +18,7 @@ mod complete_unqualified_path;
mod complete_postfix; mod complete_postfix;
mod complete_macro_in_item_position; mod complete_macro_in_item_position;
mod complete_trait_impl; mod complete_trait_impl;
mod unstable_feature_descriptor;
use ide_db::RootDatabase; use ide_db::RootDatabase;
@ -29,6 +30,11 @@ use crate::{
FilePosition, FilePosition,
}; };
//FIXME: cyclic imports caused by xtask generation, this should be better
use crate::completion::{
complete_attribute::LintCompletion, unstable_feature_descriptor::UNSTABLE_FEATURE_DESCRIPTOR,
};
pub use crate::completion::{ pub use crate::completion::{
completion_config::CompletionConfig, completion_config::CompletionConfig,
completion_item::{CompletionItem, CompletionItemKind, CompletionScore, InsertTextFormat}, completion_item::{CompletionItem, CompletionItemKind, CompletionScore, InsertTextFormat},

View file

@ -9,6 +9,7 @@ use syntax::{ast, AstNode, SyntaxKind};
use crate::completion::{ use crate::completion::{
completion_context::CompletionContext, completion_context::CompletionContext,
completion_item::{CompletionItem, CompletionItemKind, CompletionKind, Completions}, completion_item::{CompletionItem, CompletionItemKind, CompletionKind, Completions},
UNSTABLE_FEATURE_DESCRIPTOR,
}; };
pub(super) fn complete_attribute(acc: &mut Completions, ctx: &CompletionContext) -> Option<()> { pub(super) fn complete_attribute(acc: &mut Completions, ctx: &CompletionContext) -> Option<()> {
@ -17,12 +18,15 @@ pub(super) fn complete_attribute(acc: &mut Completions, ctx: &CompletionContext)
(Some(path), Some(token_tree)) if path.to_string() == "derive" => { (Some(path), Some(token_tree)) if path.to_string() == "derive" => {
complete_derive(acc, ctx, token_tree) complete_derive(acc, ctx, token_tree)
} }
(Some(path), Some(token_tree)) if path.to_string() == "feature" => {
complete_lint(acc, ctx, token_tree, UNSTABLE_FEATURE_DESCRIPTOR)
}
(Some(path), Some(token_tree)) (Some(path), Some(token_tree))
if ["allow", "warn", "deny", "forbid"] if ["allow", "warn", "deny", "forbid"]
.iter() .iter()
.any(|lint_level| lint_level == &path.to_string()) => .any(|lint_level| lint_level == &path.to_string()) =>
{ {
complete_lint(acc, ctx, token_tree) complete_lint(acc, ctx, token_tree, DEFAULT_LINT_COMPLETIONS)
} }
(_, Some(_token_tree)) => {} (_, Some(_token_tree)) => {}
_ => complete_attribute_start(acc, ctx, attribute), _ => complete_attribute_start(acc, ctx, attribute),
@ -162,9 +166,14 @@ fn complete_derive(acc: &mut Completions, ctx: &CompletionContext, derive_input:
} }
} }
fn complete_lint(acc: &mut Completions, ctx: &CompletionContext, derive_input: ast::TokenTree) { fn complete_lint(
acc: &mut Completions,
ctx: &CompletionContext,
derive_input: ast::TokenTree,
lints_completions: &[LintCompletion],
) {
if let Ok(existing_lints) = parse_comma_sep_input(derive_input) { if let Ok(existing_lints) = parse_comma_sep_input(derive_input) {
for lint_completion in DEFAULT_LINT_COMPLETIONS for lint_completion in lints_completions
.into_iter() .into_iter()
.filter(|completion| !existing_lints.contains(completion.label)) .filter(|completion| !existing_lints.contains(completion.label))
{ {
@ -228,7 +237,7 @@ fn get_derive_names_in_scope(ctx: &CompletionContext) -> FxHashSet<String> {
result result
} }
struct DeriveCompletion { pub(crate) struct DeriveCompletion {
label: &'static str, label: &'static str,
dependencies: &'static [&'static str], dependencies: &'static [&'static str],
} }
@ -248,9 +257,9 @@ const DEFAULT_DERIVE_COMPLETIONS: &[DeriveCompletion] = &[
DeriveCompletion { label: "Ord", dependencies: &["PartialOrd", "Eq", "PartialEq"] }, DeriveCompletion { label: "Ord", dependencies: &["PartialOrd", "Eq", "PartialEq"] },
]; ];
struct LintCompletion { pub(crate) struct LintCompletion {
label: &'static str, pub(crate) label: &'static str,
description: &'static str, pub(crate) description: &'static str,
} }
#[rustfmt::skip] #[rustfmt::skip]

View file

@ -29,9 +29,9 @@ pub use self::{
// Directory used by xtask // Directory used by xtask
const STORAGE: &str = ".xtask"; const STORAGE: &str = ".xtask";
const GRAMMAR_DIR: &str = "crates/ra_parser/src/grammar"; const GRAMMAR_DIR: &str = "crates/parser/src/grammar";
const OK_INLINE_TESTS_DIR: &str = "crates/ra_syntax/test_data/parser/inline/ok"; const OK_INLINE_TESTS_DIR: &str = "crates/syntax/test_data/parser/inline/ok";
const ERR_INLINE_TESTS_DIR: &str = "crates/ra_syntax/test_data/parser/inline/err"; const ERR_INLINE_TESTS_DIR: &str = "crates/syntax/test_data/parser/inline/err";
const SYNTAX_KINDS: &str = "crates/parser/src/syntax_kind/generated.rs"; const SYNTAX_KINDS: &str = "crates/parser/src/syntax_kind/generated.rs";
const AST_NODES: &str = "crates/syntax/src/ast/generated/nodes.rs"; const AST_NODES: &str = "crates/syntax/src/ast/generated/nodes.rs";
@ -41,7 +41,7 @@ const ASSISTS_DIR: &str = "crates/assists/src/handlers";
const ASSISTS_TESTS: &str = "crates/assists/src/tests/generated.rs"; const ASSISTS_TESTS: &str = "crates/assists/src/tests/generated.rs";
const REPOSITORY_URL: &str = "https://github.com/rust-lang/rust"; const REPOSITORY_URL: &str = "https://github.com/rust-lang/rust";
const UNSTABLE_FEATURE: &str = "crates/ra_ide/src/completion/unstable_feature_descriptor.rs"; const UNSTABLE_FEATURE: &str = "crates/ide/src/completion/unstable_feature_descriptor.rs";
const REPO_PATH: &str = "src/doc/unstable-book/src"; const REPO_PATH: &str = "src/doc/unstable-book/src";
#[derive(Debug, PartialEq, Eq, Clone, Copy)] #[derive(Debug, PartialEq, Eq, Clone, Copy)]

View file

@ -15,8 +15,7 @@ fn generate_descriptor(src_dir: PathBuf) -> Result<TokenStream> {
.filter_map(|e| e.ok()) .filter_map(|e| e.ok())
.filter(|entry| { .filter(|entry| {
// Get all `.md ` files // Get all `.md ` files
entry.file_type().is_file() entry.file_type().is_file() && entry.path().extension().unwrap_or_default() == "md"
&& entry.path().extension().unwrap_or_default() == "md"
}) })
.collect::<Vec<_>>(); .collect::<Vec<_>>();