2023-06-23 19:08:02 -05:00
|
|
|
use crate::assist_context::{AssistContext, Assists};
|
|
|
|
use ide_db::{assists::AssistId, SnippetCap};
|
|
|
|
use syntax::{
|
|
|
|
ast::{self, HasGenericParams, HasVisibility},
|
|
|
|
AstNode,
|
|
|
|
};
|
|
|
|
|
|
|
|
// NOTES :
|
|
|
|
// We generate erroneous code if a function is declared const (E0379)
|
|
|
|
// This is left to the user to correct as our only option is to remove the
|
|
|
|
// function completely which we should not be doing.
|
|
|
|
|
|
|
|
// Assist: generate_trait_from_impl
|
|
|
|
//
|
|
|
|
// Generate trait for an already defined inherent impl and convert impl to a trait impl.
|
|
|
|
//
|
|
|
|
// ```
|
|
|
|
// struct Foo<const N: usize>([i32; N]);
|
|
|
|
//
|
|
|
|
// macro_rules! const_maker {
|
|
|
|
// ($t:ty, $v:tt) => {
|
|
|
|
// const CONST: $t = $v;
|
|
|
|
// };
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// impl<const N: usize> Fo$0o<N> {
|
|
|
|
// // Used as an associated constant.
|
|
|
|
// const CONST_ASSOC: usize = N * 4;
|
|
|
|
//
|
|
|
|
// fn create() -> Option<()> {
|
|
|
|
// Some(())
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// const_maker! {i32, 7}
|
|
|
|
// }
|
|
|
|
// ```
|
|
|
|
// ->
|
|
|
|
// ```
|
|
|
|
// struct Foo<const N: usize>([i32; N]);
|
|
|
|
//
|
|
|
|
// macro_rules! const_maker {
|
|
|
|
// ($t:ty, $v:tt) => {
|
|
|
|
// const CONST: $t = $v;
|
|
|
|
// };
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// trait NewTrait<const N: usize> {
|
|
|
|
// // Used as an associated constant.
|
|
|
|
// const CONST_ASSOC: usize = N * 4;
|
|
|
|
//
|
|
|
|
// fn create() -> Option<()>;
|
|
|
|
//
|
|
|
|
// const_maker! {i32, 7}
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// impl<const N: usize> NewTrait<N> for Foo<N> {
|
|
|
|
// // Used as an associated constant.
|
|
|
|
// const CONST_ASSOC: usize = N * 4;
|
|
|
|
//
|
|
|
|
// fn create() -> Option<()> {
|
|
|
|
// Some(())
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// const_maker! {i32, 7}
|
|
|
|
// }
|
|
|
|
// ```
|
|
|
|
pub(crate) fn generate_trait_from_impl(acc: &mut Assists, ctx: &AssistContext<'_>) -> Option<()> {
|
|
|
|
// Get AST Node
|
|
|
|
let impl_ast = ctx.find_node_at_offset::<ast::Impl>()?;
|
|
|
|
|
|
|
|
// If impl is not inherent then we don't really need to go any further.
|
|
|
|
if impl_ast.for_token().is_some() {
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
2023-06-28 05:25:31 -05:00
|
|
|
let assoc_items = impl_ast.assoc_item_list()?;
|
2023-06-23 19:08:02 -05:00
|
|
|
let first_element = assoc_items.assoc_items().next();
|
|
|
|
if first_element.is_none() {
|
|
|
|
// No reason for an assist.
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
|
|
|
acc.add(
|
|
|
|
AssistId("generate_trait_from_impl", ide_db::assists::AssistKind::Generate),
|
|
|
|
"Generate trait from impl".to_owned(),
|
|
|
|
impl_ast.syntax().text_range(),
|
|
|
|
|builder| {
|
|
|
|
let trait_items = assoc_items.clone_for_update();
|
|
|
|
let impl_items = assoc_items.clone_for_update();
|
|
|
|
|
|
|
|
trait_items.assoc_items().for_each(|item| {
|
|
|
|
strip_body(&item);
|
|
|
|
remove_items_visibility(&item);
|
|
|
|
});
|
|
|
|
|
|
|
|
syntax::ted::replace(assoc_items.clone_for_update().syntax(), impl_items.syntax());
|
|
|
|
|
|
|
|
impl_items.assoc_items().for_each(|item| {
|
|
|
|
remove_items_visibility(&item);
|
|
|
|
});
|
|
|
|
|
|
|
|
let trait_ast = ast::make::trait_(
|
|
|
|
false,
|
|
|
|
"NewTrait".to_string(),
|
|
|
|
HasGenericParams::generic_param_list(&impl_ast),
|
|
|
|
HasGenericParams::where_clause(&impl_ast),
|
|
|
|
trait_items,
|
|
|
|
);
|
|
|
|
|
|
|
|
// Change `impl Foo` to `impl NewTrait for Foo`
|
|
|
|
// First find the PATH_TYPE which is what Foo is.
|
|
|
|
let impl_name = impl_ast.self_ty().unwrap();
|
|
|
|
let trait_name = if let Some(genpars) = impl_ast.generic_param_list() {
|
|
|
|
format!("NewTrait{}", genpars.to_generic_args())
|
|
|
|
} else {
|
|
|
|
format!("NewTrait")
|
|
|
|
};
|
|
|
|
|
|
|
|
// // Then replace
|
|
|
|
builder.replace(
|
|
|
|
impl_name.clone().syntax().text_range(),
|
|
|
|
format!("{} for {}", trait_name, impl_name.to_string()),
|
|
|
|
);
|
|
|
|
|
|
|
|
builder.replace(
|
|
|
|
impl_ast.assoc_item_list().unwrap().syntax().text_range(),
|
|
|
|
impl_items.to_string(),
|
|
|
|
);
|
|
|
|
|
|
|
|
// Insert trait before TraitImpl
|
|
|
|
builder.insert_snippet(
|
|
|
|
SnippetCap::new(true).unwrap(),
|
|
|
|
impl_ast.syntax().text_range().start(),
|
|
|
|
format!("{}\n\n", trait_ast.to_string()),
|
|
|
|
);
|
|
|
|
},
|
|
|
|
);
|
|
|
|
|
|
|
|
Some(())
|
|
|
|
}
|
|
|
|
|
|
|
|
/// `E0449` Trait items always share the visibility of their trait
|
|
|
|
fn remove_items_visibility(item: &ast::AssocItem) {
|
|
|
|
match item {
|
|
|
|
ast::AssocItem::Const(c) => {
|
|
|
|
if let Some(vis) = c.visibility() {
|
|
|
|
syntax::ted::remove(vis.syntax());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
ast::AssocItem::Fn(f) => {
|
|
|
|
if let Some(vis) = f.visibility() {
|
|
|
|
syntax::ted::remove(vis.syntax());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
ast::AssocItem::TypeAlias(t) => {
|
|
|
|
if let Some(vis) = t.visibility() {
|
|
|
|
syntax::ted::remove(vis.syntax());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
_ => (),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn strip_body(item: &ast::AssocItem) {
|
|
|
|
match item {
|
|
|
|
ast::AssocItem::Fn(f) => {
|
|
|
|
if let Some(body) = f.body() {
|
|
|
|
// In constrast to function bodies, we want to see no ws before a semicolon.
|
|
|
|
// So let's remove them if we see any.
|
|
|
|
if let Some(prev) = body.syntax().prev_sibling_or_token() {
|
|
|
|
if prev.kind() == syntax::SyntaxKind::WHITESPACE {
|
|
|
|
syntax::ted::remove(prev);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
syntax::ted::replace(body.syntax(), ast::make::tokens::semicolon());
|
|
|
|
}
|
|
|
|
}
|
|
|
|
_ => (),
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
#[cfg(test)]
|
|
|
|
mod tests {
|
|
|
|
use super::*;
|
|
|
|
use crate::tests::{check_assist, check_assist_not_applicable};
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_assoc_item_fn() {
|
|
|
|
check_assist(
|
|
|
|
generate_trait_from_impl,
|
|
|
|
r#"
|
|
|
|
struct Foo(f64);
|
|
|
|
|
|
|
|
impl F$0oo {
|
|
|
|
fn add(&mut self, x: f64) {
|
|
|
|
self.0 += x;
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
struct Foo(f64);
|
|
|
|
|
|
|
|
trait NewTrait {
|
|
|
|
fn add(&mut self, x: f64);
|
|
|
|
}
|
|
|
|
|
|
|
|
impl NewTrait for Foo {
|
|
|
|
fn add(&mut self, x: f64) {
|
|
|
|
self.0 += x;
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_assoc_item_macro() {
|
|
|
|
check_assist(
|
|
|
|
generate_trait_from_impl,
|
|
|
|
r#"
|
|
|
|
struct Foo;
|
|
|
|
|
|
|
|
macro_rules! const_maker {
|
|
|
|
($t:ty, $v:tt) => {
|
|
|
|
const CONST: $t = $v;
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
impl F$0oo {
|
|
|
|
const_maker! {i32, 7}
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
struct Foo;
|
|
|
|
|
|
|
|
macro_rules! const_maker {
|
|
|
|
($t:ty, $v:tt) => {
|
|
|
|
const CONST: $t = $v;
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
trait NewTrait {
|
|
|
|
const_maker! {i32, 7}
|
|
|
|
}
|
|
|
|
|
|
|
|
impl NewTrait for Foo {
|
|
|
|
const_maker! {i32, 7}
|
|
|
|
}"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_assoc_item_const() {
|
|
|
|
check_assist(
|
|
|
|
generate_trait_from_impl,
|
|
|
|
r#"
|
|
|
|
struct Foo;
|
|
|
|
|
|
|
|
impl F$0oo {
|
|
|
|
const ABC: i32 = 3;
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
struct Foo;
|
|
|
|
|
|
|
|
trait NewTrait {
|
|
|
|
const ABC: i32 = 3;
|
|
|
|
}
|
|
|
|
|
|
|
|
impl NewTrait for Foo {
|
|
|
|
const ABC: i32 = 3;
|
|
|
|
}"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_impl_with_generics() {
|
|
|
|
check_assist(
|
|
|
|
generate_trait_from_impl,
|
|
|
|
r#"
|
|
|
|
struct Foo<const N: usize>([i32; N]);
|
|
|
|
|
|
|
|
impl<const N: usize> F$0oo<N> {
|
|
|
|
// Used as an associated constant.
|
|
|
|
const CONST: usize = N * 4;
|
|
|
|
}
|
|
|
|
"#,
|
|
|
|
r#"
|
|
|
|
struct Foo<const N: usize>([i32; N]);
|
|
|
|
|
|
|
|
trait NewTrait<const N: usize> {
|
|
|
|
// Used as an associated constant.
|
|
|
|
const CONST: usize = N * 4;
|
|
|
|
}
|
|
|
|
|
|
|
|
impl<const N: usize> NewTrait<N> for Foo<N> {
|
|
|
|
// Used as an associated constant.
|
|
|
|
const CONST: usize = N * 4;
|
|
|
|
}
|
|
|
|
"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_e0449_avoided() {
|
|
|
|
check_assist(
|
|
|
|
generate_trait_from_impl,
|
|
|
|
r#"
|
|
|
|
struct Foo;
|
|
|
|
|
|
|
|
impl F$0oo {
|
|
|
|
pub fn a_func() -> Option<()> {
|
|
|
|
Some(())
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
struct Foo;
|
|
|
|
|
|
|
|
trait NewTrait {
|
|
|
|
fn a_func() -> Option<()>;
|
|
|
|
}
|
|
|
|
|
|
|
|
impl NewTrait for Foo {
|
|
|
|
fn a_func() -> Option<()> {
|
|
|
|
Some(())
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn test_empty_inherent_impl() {
|
|
|
|
check_assist_not_applicable(
|
|
|
|
generate_trait_from_impl,
|
|
|
|
r#"
|
|
|
|
impl Emp$0tyImpl{}
|
|
|
|
"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
}
|