From 9bc5142fce91223c4ec1760203135aa7e960f09e Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net>
Date: Tue, 13 May 2025 11:38:17 +0200
Subject: [PATCH] client: Port "player" menu to pre_run

---
 client/main.c   | 1 -
 client/player.c | 4 +++-
 client/player.h | 1 -
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/client/main.c b/client/main.c
index 5d32b5218..ce753c1ac 100644
--- a/client/main.c
+++ b/client/main.c
@@ -3426,7 +3426,6 @@ int main(int argc, char *argv[])
 		bt_shell_set_env("AUTO_REGISTER_ENDPOINT",
 					(void *)endpoint_option);
 
-	player_enable_submenu();
 	mgmt_enable_submenu();
 	assistant_enable_submenu();
 
diff --git a/client/player.c b/client/player.c
index 7aab4af6d..0314cd303 100644
--- a/client/player.c
+++ b/client/player.c
@@ -154,6 +154,7 @@ struct transport_select_args {
 	struct queue *selecting;
 };
 
+static void player_menu_pre_run(const struct bt_shell_menu *menu);
 static void transport_set_links(struct transport_select_args *args);
 static void transport_select(struct transport_select_args *args);
 
@@ -5838,6 +5839,7 @@ static void cmd_volume_transport(int argc, char *argv[])
 static const struct bt_shell_menu transport_menu = {
 	.name = "transport",
 	.desc = "Media Transport Submenu",
+	.pre_run = player_menu_pre_run,
 	.entries = {
 	{ "list",         NULL,    cmd_list_transport,
 						"List available transports" },
@@ -5878,7 +5880,7 @@ void player_add_submenu(void)
 	bt_shell_add_submenu(&transport_menu);
 }
 
-void player_enable_submenu(void)
+static void player_menu_pre_run(const struct bt_shell_menu *menu)
 {
 	dbus_conn = bt_shell_get_env("DBUS_CONNECTION");
 	if (!dbus_conn || client)
diff --git a/client/player.h b/client/player.h
index c09ffa94c..e7778cb1e 100644
--- a/client/player.h
+++ b/client/player.h
@@ -9,5 +9,4 @@
  */
 
 void player_add_submenu(void);
-void player_enable_submenu(void);
 void player_remove_submenu(void);
-- 
2.47.3