From 54c8c39da0010d05a5f1add2190398dff03fca79 Mon Sep 17 00:00:00 2001
From: Jonas Schievink <jonas.schievink@ferrous-systems.com>
Date: Mon, 16 May 2022 19:27:27 +0200
Subject: [PATCH] Don't generate documentation in `generate_setter`

---
 .../ide-assists/src/handlers/generate_setter.rs  | 16 +---------------
 crates/ide-assists/src/tests/generated.rs        |  1 -
 2 files changed, 1 insertion(+), 16 deletions(-)

diff --git a/crates/ide-assists/src/handlers/generate_setter.rs b/crates/ide-assists/src/handlers/generate_setter.rs
index 5005ac9f5c..62576a854f 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 099d7a43a7..5a3f23309e 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;
     }