From 183a38fb50f284de1ca02c05ed945e240f3c0274 Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Thu, 26 Sep 2019 12:18:26 +0300 Subject: keep ast creation API simple --- .../src/assists/add_missing_impl_members.rs | 7 +- crates/ra_assists/src/assists/fill_match_arms.rs | 27 +-- crates/ra_assists/src/assists/move_bounds.rs | 10 +- crates/ra_assists/src/ast_builder.rs | 218 --------------------- crates/ra_assists/src/ast_editor.rs | 52 ++++- crates/ra_assists/src/lib.rs | 4 +- 6 files changed, 74 insertions(+), 244 deletions(-) delete mode 100644 crates/ra_assists/src/ast_builder.rs (limited to 'crates/ra_assists') diff --git a/crates/ra_assists/src/assists/add_missing_impl_members.rs b/crates/ra_assists/src/assists/add_missing_impl_members.rs index 22d20909d..23da1e65f 100644 --- a/crates/ra_assists/src/assists/add_missing_impl_members.rs +++ b/crates/ra_assists/src/assists/add_missing_impl_members.rs @@ -1,10 +1,10 @@ use hir::{db::HirDatabase, HasSource}; use ra_syntax::{ - ast::{self, AstNode, NameOwner}, + ast::{self, make, AstNode, NameOwner}, SmolStr, }; -use crate::{ast_builder::Make, ast_editor::AstEditor, Assist, AssistCtx, AssistId}; +use crate::{ast_editor::AstEditor, Assist, AssistCtx, AssistId}; #[derive(PartialEq)] enum AddMissingImplMembersMode { @@ -102,7 +102,8 @@ fn strip_docstring(item: ast::ImplItem) -> ast::ImplItem { fn add_body(fn_def: ast::FnDef) -> ast::FnDef { let mut ast_editor = AstEditor::new(fn_def.clone()); if fn_def.body().is_none() { - ast_editor.set_body(&Make::::single_expr(Make::::unimplemented())); + let body = make::block_from_expr(make::expr_unimplemented()); + ast_editor.set_body(&body); } ast_editor.ast().to_owned() } diff --git a/crates/ra_assists/src/assists/fill_match_arms.rs b/crates/ra_assists/src/assists/fill_match_arms.rs index 817433526..db82db89a 100644 --- a/crates/ra_assists/src/assists/fill_match_arms.rs +++ b/crates/ra_assists/src/assists/fill_match_arms.rs @@ -1,9 +1,9 @@ use std::iter; use hir::{db::HirDatabase, Adt, HasSource}; -use ra_syntax::ast::{self, AstNode, NameOwner}; +use ra_syntax::ast::{self, make, AstNode, NameOwner}; -use crate::{ast_builder::Make, Assist, AssistCtx, AssistId}; +use crate::{Assist, AssistCtx, AssistId}; pub(crate) fn fill_match_arms(mut ctx: AssistCtx) -> Option { let match_expr = ctx.node_at_offset::()?; @@ -31,8 +31,8 @@ pub(crate) fn fill_match_arms(mut ctx: AssistCtx) -> Option::from(iter::once(pat), Make::::unit())); - let new_arm_list = Make::::from_arms(arms); + .map(|pat| make::match_arm(iter::once(pat), make::expr_unit())); + let new_arm_list = make::match_arm_list(arms); edit.target(match_expr.syntax().text_range()); edit.set_cursor(expr.syntax().text_range().start()); @@ -63,21 +63,22 @@ fn resolve_enum_def( } fn build_pat(var: ast::EnumVariant) -> Option { - let path = Make::::from(var.parent_enum().name()?, var.name()?); + let path = make::path_qualified( + make::path_from_name_ref(make::name_ref(&var.parent_enum().name()?.syntax().to_string())), + make::name_ref(&var.name()?.syntax().to_string()), + ); let pat: ast::Pat = match var.kind() { ast::StructKind::Tuple(field_list) => { - let pats = iter::repeat(Make::::placeholder().into()) - .take(field_list.fields().count()); - Make::::from(path, pats).into() + let pats = + iter::repeat(make::placeholder_pat().into()).take(field_list.fields().count()); + make::tuple_struct_pat(path, pats).into() } ast::StructKind::Named(field_list) => { - let pats = field_list - .fields() - .map(|f| Make::::from_name(f.name().unwrap()).into()); - Make::::from(path, pats).into() + let pats = field_list.fields().map(|f| make::bind_pat(f.name().unwrap()).into()); + make::record_pat(path, pats).into() } - ast::StructKind::Unit => Make::::from_path(path).into(), + ast::StructKind::Unit => make::path_pat(path).into(), }; Some(pat) diff --git a/crates/ra_assists/src/assists/move_bounds.rs b/crates/ra_assists/src/assists/move_bounds.rs index 671826013..fd4bdc55c 100644 --- a/crates/ra_assists/src/assists/move_bounds.rs +++ b/crates/ra_assists/src/assists/move_bounds.rs @@ -1,11 +1,11 @@ use hir::db::HirDatabase; use ra_syntax::{ - ast::{self, AstNode, NameOwner, TypeBoundsOwner}, + ast::{self, make, AstNode, NameOwner, TypeBoundsOwner}, SyntaxElement, SyntaxKind::*, }; -use crate::{ast_builder::Make, ast_editor::AstEditor, Assist, AssistCtx, AssistId}; +use crate::{ast_editor::AstEditor, Assist, AssistCtx, AssistId}; pub(crate) fn move_bounds_to_where_clause(mut ctx: AssistCtx) -> Option { let type_param_list = ctx.node_at_offset::()?; @@ -50,7 +50,7 @@ pub(crate) fn move_bounds_to_where_clause(mut ctx: AssistCtx) let where_clause = { let predicates = type_param_list.type_params().filter_map(build_predicate); - Make::::from_predicates(predicates) + make::where_clause(predicates) }; let to_insert = match anchor.prev_sibling_or_token() { @@ -68,8 +68,8 @@ pub(crate) fn move_bounds_to_where_clause(mut ctx: AssistCtx) } fn build_predicate(param: ast::TypeParam) -> Option { - let path = Make::::from_name(param.name()?); - let predicate = Make::::from(path, param.type_bound_list()?.bounds()); + let path = make::path_from_name_ref(make::name_ref(¶m.name()?.syntax().to_string())); + let predicate = make::where_pred(path, param.type_bound_list()?.bounds()); Some(predicate) } diff --git a/crates/ra_assists/src/ast_builder.rs b/crates/ra_assists/src/ast_builder.rs deleted file mode 100644 index 9a62b96b3..000000000 --- a/crates/ra_assists/src/ast_builder.rs +++ /dev/null @@ -1,218 +0,0 @@ -use itertools::Itertools; - -use ra_syntax::{ast, AstNode, SourceFile}; - -pub struct Make { - _phantom: std::marker::PhantomData, -} - -impl Make { - pub fn from(name: ast::NameRef, expr: Option) -> ast::RecordField { - match expr { - Some(expr) => Self::from_text(&format!("{}: {}", name.syntax(), expr.syntax())), - None => Self::from_text(&name.syntax().to_string()), - } - } - - fn from_text(text: &str) -> ast::RecordField { - ast_node_from_file_text(&format!("fn f() {{ S {{ {}, }} }}", text)) - } -} - -impl Make { - pub fn single_expr(e: ast::Expr) -> ast::Block { - Self::from_text(&format!("{{ {} }}", e.syntax())) - } - - fn from_text(text: &str) -> ast::Block { - ast_node_from_file_text(&format!("fn f() {}", text)) - } -} - -impl Make { - pub fn unit() -> ast::Expr { - Self::from_text("()") - } - - pub fn unimplemented() -> ast::Expr { - Self::from_text("unimplemented!()") - } - - fn from_text(text: &str) -> ast::Expr { - ast_node_from_file_text(&format!("const C: () = {};", text)) - } -} - -impl Make { - pub fn from(text: &str) -> ast::NameRef { - ast_node_from_file_text(&format!("fn f() {{ {}; }}", text)) - } -} - -impl Make { - pub fn from_name(name: ast::Name) -> ast::Path { - let name = name.syntax().to_string(); - Self::from_text(name.as_str()) - } - - pub fn from(enum_name: ast::Name, var_name: ast::Name) -> ast::Path { - Self::from_text(&format!("{}::{}", enum_name.syntax(), var_name.syntax())) - } - - fn from_text(text: &str) -> ast::Path { - ast_node_from_file_text(text) - } -} - -impl Make { - pub fn from_name(name: ast::Name) -> ast::BindPat { - Self::from_text(name.text()) - } - - fn from_text(text: &str) -> ast::BindPat { - ast_node_from_file_text(&format!("fn f({}: ())", text)) - } -} - -impl Make { - pub fn placeholder() -> ast::PlaceholderPat { - Self::from_text("_") - } - - fn from_text(text: &str) -> ast::PlaceholderPat { - ast_node_from_file_text(&format!("fn f({}: ())", text)) - } -} - -impl Make { - pub fn from(path: ast::Path, pats: impl Iterator) -> ast::TupleStructPat { - let pats_str = pats.map(|p| p.syntax().to_string()).collect::>().join(", "); - Self::from_text(&format!("{}({})", path.syntax(), pats_str)) - } - - fn from_text(text: &str) -> ast::TupleStructPat { - ast_node_from_file_text(&format!("fn f({}: ())", text)) - } -} - -impl Make { - pub fn from(path: ast::Path, pats: impl Iterator) -> ast::RecordPat { - let pats_str = pats.map(|p| p.syntax().to_string()).collect::>().join(", "); - Self::from_text(&format!("{}{{ {} }}", path.syntax(), pats_str)) - } - - fn from_text(text: &str) -> ast::RecordPat { - ast_node_from_file_text(&format!("fn f({}: ())", text)) - } -} - -impl Make { - pub fn from_path(path: ast::Path) -> ast::PathPat { - let path_str = path.syntax().text().to_string(); - Self::from_text(path_str.as_str()) - } - - fn from_text(text: &str) -> ast::PathPat { - ast_node_from_file_text(&format!("fn f({}: ())", text)) - } -} - -impl Make { - pub fn from(pats: impl Iterator, expr: ast::Expr) -> ast::MatchArm { - let pats_str = pats.map(|p| p.syntax().to_string()).join(" | "); - Self::from_text(&format!("{} => {}", pats_str, expr.syntax())) - } - - fn from_text(text: &str) -> ast::MatchArm { - ast_node_from_file_text(&format!("fn f() {{ match () {{{}}} }}", text)) - } -} - -impl Make { - pub fn from_arms(arms: impl Iterator) -> ast::MatchArmList { - let arms_str = arms.map(|arm| format!("\n {}", arm.syntax())).join(","); - Self::from_text(&format!("{},\n", arms_str)) - } - - fn from_text(text: &str) -> ast::MatchArmList { - ast_node_from_file_text(&format!("fn f() {{ match () {{{}}} }}", text)) - } -} - -impl Make { - pub fn from(path: ast::Path, bounds: impl Iterator) -> ast::WherePred { - let bounds = bounds.map(|b| b.syntax().to_string()).collect::>().join(" + "); - Self::from_text(&format!("{}: {}", path.syntax(), bounds)) - } - - fn from_text(text: &str) -> ast::WherePred { - ast_node_from_file_text(&format!("fn f() where {} {{ }}", text)) - } -} - -impl Make { - pub fn from_predicates(preds: impl Iterator) -> ast::WhereClause { - let preds = preds.map(|p| p.syntax().to_string()).collect::>().join(", "); - Self::from_text(preds.as_str()) - } - - fn from_text(text: &str) -> ast::WhereClause { - ast_node_from_file_text(&format!("fn f() where {} {{ }}", text)) - } -} - -fn ast_node_from_file_text(text: &str) -> N { - let parse = SourceFile::parse(text); - let res = parse.tree().syntax().descendants().find_map(N::cast).unwrap(); - res -} - -pub(crate) mod tokens { - use once_cell::sync::Lazy; - use ra_syntax::{AstNode, Parse, SourceFile, SyntaxKind::*, SyntaxToken, T}; - - static SOURCE_FILE: Lazy> = Lazy::new(|| SourceFile::parse(",\n; ;")); - - pub(crate) fn comma() -> SyntaxToken { - SOURCE_FILE - .tree() - .syntax() - .descendants_with_tokens() - .filter_map(|it| it.into_token()) - .find(|it| it.kind() == T![,]) - .unwrap() - } - - pub(crate) fn single_space() -> SyntaxToken { - SOURCE_FILE - .tree() - .syntax() - .descendants_with_tokens() - .filter_map(|it| it.into_token()) - .find(|it| it.kind() == WHITESPACE && it.text().as_str() == " ") - .unwrap() - } - - #[allow(unused)] - pub(crate) fn single_newline() -> SyntaxToken { - SOURCE_FILE - .tree() - .syntax() - .descendants_with_tokens() - .filter_map(|it| it.into_token()) - .find(|it| it.kind() == WHITESPACE && it.text().as_str() == "\n") - .unwrap() - } - - pub(crate) struct WsBuilder(SourceFile); - - impl WsBuilder { - pub(crate) fn new(text: &str) -> WsBuilder { - WsBuilder(SourceFile::parse(text).ok().unwrap()) - } - pub(crate) fn ws(&self) -> SyntaxToken { - self.0.syntax().first_child_or_token().unwrap().into_token().unwrap() - } - } - -} diff --git a/crates/ra_assists/src/ast_editor.rs b/crates/ra_assists/src/ast_editor.rs index 4e253f0a4..2936c094b 100644 --- a/crates/ra_assists/src/ast_editor.rs +++ b/crates/ra_assists/src/ast_editor.rs @@ -13,8 +13,6 @@ use ra_syntax::{ }; use ra_text_edit::TextEditBuilder; -use crate::ast_builder::tokens; - pub struct AstEditor { original_ast: N, ast: N, @@ -286,3 +284,53 @@ impl AstEditor { self } } + +mod tokens { + use once_cell::sync::Lazy; + use ra_syntax::{AstNode, Parse, SourceFile, SyntaxKind::*, SyntaxToken, T}; + + static SOURCE_FILE: Lazy> = Lazy::new(|| SourceFile::parse(",\n; ;")); + + pub(crate) fn comma() -> SyntaxToken { + SOURCE_FILE + .tree() + .syntax() + .descendants_with_tokens() + .filter_map(|it| it.into_token()) + .find(|it| it.kind() == T![,]) + .unwrap() + } + + pub(crate) fn single_space() -> SyntaxToken { + SOURCE_FILE + .tree() + .syntax() + .descendants_with_tokens() + .filter_map(|it| it.into_token()) + .find(|it| it.kind() == WHITESPACE && it.text().as_str() == " ") + .unwrap() + } + + #[allow(unused)] + pub(crate) fn single_newline() -> SyntaxToken { + SOURCE_FILE + .tree() + .syntax() + .descendants_with_tokens() + .filter_map(|it| it.into_token()) + .find(|it| it.kind() == WHITESPACE && it.text().as_str() == "\n") + .unwrap() + } + + pub(crate) struct WsBuilder(SourceFile); + + impl WsBuilder { + pub(crate) fn new(text: &str) -> WsBuilder { + WsBuilder(SourceFile::parse(text).ok().unwrap()) + } + pub(crate) fn ws(&self) -> SyntaxToken { + self.0.syntax().first_child_or_token().unwrap().into_token().unwrap() + } + } + +} diff --git a/crates/ra_assists/src/lib.rs b/crates/ra_assists/src/lib.rs index 71b017076..897af2b02 100644 --- a/crates/ra_assists/src/lib.rs +++ b/crates/ra_assists/src/lib.rs @@ -8,11 +8,9 @@ mod assist_ctx; mod marks; pub mod ast_editor; -pub mod ast_builder; - -use itertools::Itertools; use hir::db::HirDatabase; +use itertools::Itertools; use ra_db::FileRange; use ra_syntax::{TextRange, TextUnit}; use ra_text_edit::TextEdit; -- cgit v1.2.3