On Mon, 2016-01-18 at 16:38 +0000, Frediano Ziglio wrote: > From: Jonathon Jongsma <jjong...@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[] = { _______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel