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