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

Pavel Grunt pgrunt at redhat.com
Tue Jan 19 04:24:58 PST 2016


On Tue, 2016-01-19 at 10:03 +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.

Acked-by: Pavel Grunt <pgrunt at redhat.com>
> ---
>  server/reds.c | 4 ++--
>  server/reds.h | 1 -
>  2 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/server/reds.c b/server/reds.c
> index 0c39c05..22ab26d 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -897,7 +897,7 @@ int reds_get_n_channels(RedsState *reds)
>  }
>  
>  
> -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 @@ 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[] = {
> diff --git a/server/reds.h b/server/reds.h
> index 99fd97f..39fe291 100644
> --- a/server/reds.h
> +++ b/server/reds.h
> @@ -88,7 +88,6 @@ typedef struct MainMigrateData MainMigrateData;
>  void reds_marshall_migrate_data(SpiceMarshaller *m);
>  void reds_fill_channels(RedsState *reds, SpiceMsgChannels
> *channels_info);
>  int reds_get_n_channels(RedsState *reds);
> -int reds_num_of_clients(RedsState *reds);
>  #ifdef RED_STATISTICS
>  void reds_update_stat_value(uint32_t value);
>  #endif


More information about the Spice-devel mailing list