Re: [PATCH 15/15] reds_num_of_clients() -> reds_get_n_clients()

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

 



On Mon, 2016-01-18 at 16:18 -0600, Jonathon Jongsma wrote:
> On Mon, 2016-01-18 at 16:38 +0000, Frediano Ziglio wrote:
> > From: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
> > 
> > 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.
Yes, it should be moved to the PATCH 07

Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx>

> 
> > ---
> >  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[] = {
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]