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