[PATCH spice-server] reds: Inline very simple function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



reds_get_n_clients is a single line and is used only by
spice_server_get_num_clients.
The 2 functions have very similar names so inlining
reds_get_n_clients does not make code less readable.

Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
---
 server/reds.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/server/reds.c b/server/reds.c
index 84c2e466..a738afc3 100644
--- a/server/reds.c
+++ b/server/reds.c
@@ -924,14 +924,9 @@ void reds_handle_agent_mouse_event(RedsState *reds, const VDAgentMouseState *mou
     red_char_device_write_buffer_add(RED_CHAR_DEVICE(reds->agent_dev), char_dev_buf);
 }
 
-static int reds_get_n_clients(RedsState *reds)
-{
-    return reds ? g_list_length(reds->clients) : 0;
-}
-
 SPICE_GNUC_VISIBLE int spice_server_get_num_clients(SpiceServer *reds)
 {
-    return reds_get_n_clients(reds);
+    return reds ? g_list_length(reds->clients) : 0;
 }
 
 static bool channel_supports_multiple_clients(RedChannel *channel)
-- 
2.13.5

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]