client/player: Fix checkpatch warning
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Tue, 30 Aug 2022 21:24:08 +0000 (14:24 -0700)
committerAyush Garg <ayush.garg@samsung.com>
Mon, 15 May 2023 09:25:54 +0000 (14:55 +0530)
This fixes the following checkpatch warning:

WARNING:LINE_SPACING: Missing a blank line after declarations
216: FILE: client/player.c:625:
+               GDBusProxy *proxy = l->data;
+               print_player(proxy, NULL);

Signed-off-by: Manika Shrivastava <manika.sh@samsung.com>
Signed-off-by: Ayush Garg <ayush.garg@samsung.com>
client/player.c

index aa6dff4..07d1d20 100644 (file)
@@ -603,8 +603,10 @@ static void print_media(GDBusProxy *proxy, const char *description)
        g_free(str);
 }
 
-static void print_player(GDBusProxy *proxy, const char *description)
+static void print_player(void *data, void *user_data)
 {
+       GDBusProxy *proxy = data;
+       const char *description = user_data;
        char *str;
 
        str = proxy_description(proxy, "Player", description);
@@ -617,12 +619,7 @@ static void print_player(GDBusProxy *proxy, const char *description)
 
 static void cmd_list(int argc, char *arg[])
 {
-       GList *l;
-
-       for (l = players; l; l = g_list_next(l)) {
-               GDBusProxy *proxy = l->data;
-               print_player(proxy, NULL);
-       }
+       g_list_foreach(players, print_player, NULL);
 
        return bt_shell_noninteractive_quit(EXIT_SUCCESS);
 }