about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJonas Schievink <jonas.schievink@ferrous-systems.com>2022-05-16 19:27:27 +0200
committerJonas Schievink <jonas.schievink@ferrous-systems.com>2022-05-16 19:27:27 +0200
commit54c8c39da0010d05a5f1add2190398dff03fca79 (patch)
tree37f6166d5566e9db3e0401d0edbadac324426f91
parentda503b6a1349b1402bfc9e2d109c7c16b3e9b9f4 (diff)
downloadrust-54c8c39da0010d05a5f1add2190398dff03fca79.tar.gz
rust-54c8c39da0010d05a5f1add2190398dff03fca79.zip
Don't generate documentation in `generate_setter`
-rw-r--r--crates/ide-assists/src/handlers/generate_setter.rs16
-rw-r--r--crates/ide-assists/src/tests/generated.rs1
2 files changed, 1 insertions, 16 deletions
diff --git a/crates/ide-assists/src/handlers/generate_setter.rs b/crates/ide-assists/src/handlers/generate_setter.rs
index 5005ac9f5c1..62576a854f9 100644
--- a/crates/ide-assists/src/handlers/generate_setter.rs
+++ b/crates/ide-assists/src/handlers/generate_setter.rs
@@ -22,7 +22,6 @@ use crate::{
 // }
 //
 // impl Person {
-//     /// Set the person's name.
 //     fn set_name(&mut self, name: String) {
 //         self.name = name;
 //     }
@@ -32,7 +31,6 @@ pub(crate) fn generate_setter(acc: &mut Assists, ctx: &AssistContext) -> Option<
     let strukt = ctx.find_node_at_offset::<ast::Struct>()?;
     let field = ctx.find_node_at_offset::<ast::RecordField>()?;
 
-    let strukt_name = strukt.name()?;
     let field_name = field.name()?;
     let field_ty = field.ty()?;
 
@@ -53,10 +51,6 @@ pub(crate) fn generate_setter(acc: &mut Assists, ctx: &AssistContext) -> Option<
         |builder| {
             let mut buf = String::with_capacity(512);
 
-            let fn_name_spaced = fn_name.replace('_', " ");
-            let strukt_name_spaced =
-                to_lower_snake_case(&strukt_name.to_string()).replace('_', " ");
-
             if impl_def.is_some() {
                 buf.push('\n');
             }
@@ -64,12 +58,9 @@ pub(crate) fn generate_setter(acc: &mut Assists, ctx: &AssistContext) -> Option<
             let vis = strukt.visibility().map_or(String::new(), |v| format!("{} ", v));
             format_to!(
                 buf,
-                "    /// Set the {}'s {}.
-    {}fn set_{}(&mut self, {}: {}) {{
+                "    {}fn set_{}(&mut self, {}: {}) {{
         self.{} = {};
     }}",
-                strukt_name_spaced,
-                fn_name_spaced,
                 vis,
                 fn_name,
                 fn_name,
@@ -114,7 +105,6 @@ struct Person<T: Clone> {
 }
 
 impl<T: Clone> Person<T> {
-    /// Set the person's data.
     fn set_data(&mut self, data: T) {
         self.data = data;
     }
@@ -152,7 +142,6 @@ pub(crate) struct Person<T: Clone> {
 }
 
 impl<T: Clone> Person<T> {
-    /// Set the person's data.
     pub(crate) fn set_data(&mut self, data: T) {
         self.data = data;
     }
@@ -171,7 +160,6 @@ struct Context<T: Clone> {
 }
 
 impl<T: Clone> Context<T> {
-    /// Set the context's data.
     fn set_data(&mut self, data: T) {
         self.data = data;
     }
@@ -183,12 +171,10 @@ struct Context<T: Clone> {
 }
 
 impl<T: Clone> Context<T> {
-    /// Set the context's data.
     fn set_data(&mut self, data: T) {
         self.data = data;
     }
 
-    /// Set the context's count.
     fn set_count(&mut self, count: usize) {
         self.count = count;
     }
diff --git a/crates/ide-assists/src/tests/generated.rs b/crates/ide-assists/src/tests/generated.rs
index 099d7a43a7f..5a3f23309e8 100644
--- a/crates/ide-assists/src/tests/generated.rs
+++ b/crates/ide-assists/src/tests/generated.rs
@@ -1158,7 +1158,6 @@ struct Person {
 }
 
 impl Person {
-    /// Set the person's name.
     fn set_name(&mut self, name: String) {
         self.name = name;
     }