[Spice-devel] [PATCH 15/15] reds_num_of_clients() -> reds_get_n_clients()

Jonathon Jongsma jjongsma at redhat.com
Mon Jan 18 14:18:33 PST 2016


On Mon, 2016-01-18 at 16:38 +0000, Frediano Ziglio wrote:
> From: Jonathon Jongsma <jjongsma at redhat.com>
> 
> More consistent with glib naming conventions. Also make the function
> static since it's not used outside of this source file.

I don't know if this patch has changed slightly due to rebase, but the function
appears to already be static. So we can remove this last sentence from the
commit log.

> ---
>  server/reds.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/server/reds.c b/server/reds.c
> index 4de1640..22ab26d 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -897,7 +897,7 @@ int reds_get_n_channels(RedsState *reds)
>  }
>  
>  
> -static int reds_num_of_clients(RedsState *reds)
> +static int reds_get_n_clients(RedsState *reds)
>  {
>      return reds ? reds->num_clients : 0;
>  }
> @@ -905,7 +905,7 @@ static int reds_num_of_clients(RedsState *reds)
>  SPICE_GNUC_VISIBLE int spice_server_get_num_clients(SpiceServer *s)
>  {
>      spice_assert(reds == s);
> -    return reds_num_of_clients(reds);
> +    return reds_get_n_clients(reds);
>  }
>  
>  static int secondary_channels[] = {


More information about the Spice-devel mailing list