From 7f04f87834670cbabe564055dc1cfb086b1af241 Mon Sep 17 00:00:00 2001
From: Gnome! <david2005thomas@Gmail.com>
Date: Mon, 7 Oct 2024 23:11:59 +0100
Subject: [PATCH] Swap `Vec<T>` for `Cow<[T]>` in `CreateActionRow::Buttons`
 (#2985)

Closes #2984.
---
 src/builder/create_components.rs | 16 +++++++++++++++-
 src/builder/mod.rs               |  4 ++--
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/src/builder/create_components.rs b/src/builder/create_components.rs
index 10be84027f9..516f7aeac00 100644
--- a/src/builder/create_components.rs
+++ b/src/builder/create_components.rs
@@ -11,12 +11,26 @@ use crate::model::prelude::*;
 #[derive(Clone, Debug)]
 #[must_use]
 pub enum CreateActionRow<'a> {
-    Buttons(Vec<CreateButton<'a>>),
+    Buttons(Cow<'a, [CreateButton<'a>]>),
     SelectMenu(CreateSelectMenu<'a>),
     /// Only valid in modals!
     InputText(CreateInputText<'a>),
 }
 
+impl<'a> CreateActionRow<'a> {
+    pub fn buttons(buttons: impl Into<Cow<'a, [CreateButton<'a>]>>) -> Self {
+        Self::Buttons(buttons.into())
+    }
+
+    pub fn select_menu(select_menu: impl Into<CreateSelectMenu<'a>>) -> Self {
+        Self::SelectMenu(select_menu.into())
+    }
+
+    pub fn input_text(input_text: impl Into<CreateInputText<'a>>) -> Self {
+        Self::InputText(input_text.into())
+    }
+}
+
 impl<'a> serde::Serialize for CreateActionRow<'a> {
     fn serialize<S: serde::Serializer>(&self, serializer: S) -> Result<S::Ok, S::Error> {
         use serde::ser::Error as _;
diff --git a/src/builder/mod.rs b/src/builder/mod.rs
index 9f63d353611..8050f6718a9 100644
--- a/src/builder/mod.rs
+++ b/src/builder/mod.rs
@@ -128,12 +128,12 @@ macro_rules! button_and_select_menu_convenience_methods {
         pub fn button(mut $self, button: super::CreateButton<'a>) -> Self {
             let rows = $self$(.$components_path)+.get_or_insert_with(Cow::default).to_mut();
             let row_with_space_left = rows.last_mut().and_then(|row| match row {
-                super::CreateActionRow::Buttons(buttons) if buttons.len() < 5 => Some(buttons),
+                super::CreateActionRow::Buttons(buttons) if buttons.len() < 5 => Some(buttons.to_mut()),
                 _ => None,
             });
             match row_with_space_left {
                 Some(row) => row.push(button),
-                None => rows.push(super::CreateActionRow::Buttons(vec![button])),
+                None => rows.push(super::CreateActionRow::buttons(vec![button])),
             }
             $self
         }