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

Frediano Ziglio fziglio at redhat.com
Tue Jan 19 02:04:27 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.
> 

My fault. Posted new versions for 07/15 and 15/15

Frediano

> > ---
> >  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