Re: [Spice-devel] [spice-server 03/17] sound: Remove extraneous whitespace

2017-01-11 Thread Christophe Fergeau 
On Wed, Jan 11, 2017 at 05:50:43AM -0500, Frediano Ziglio wrote:
> > 
> > No need for this whitespace before ';'
> > ---
> >  server/sound.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/server/sound.c b/server/sound.c
> > index f154465..1f6c243 100644
> > --- a/server/sound.c
> > +++ b/server/sound.c
> > @@ -1508,7 +1508,7 @@ static void on_new_record_channel_client(SndChannel
> > *channel, SndChannelClient *
> >  {
> >  spice_assert(client);
> >  
> > -channel->connection = client ;
> > +channel->connection = client;
> >  if (channel->volume.volume_nchannels) {
> >  snd_set_command(client, SND_VOLUME_MASK);
> >  }
> 
> Acked-by: Frediano Ziglio 
> 
> A bit picky I would say.

This was part of that big patch, but did not really fit well anywhere,
so I just split it.

Christophe


signature.asc
Description: PGP signature
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Re: [Spice-devel] [spice-server 03/17] sound: Remove extraneous whitespace

2017-01-11 Thread Frediano Ziglio 
> 
> No need for this whitespace before ';'
> ---
>  server/sound.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/server/sound.c b/server/sound.c
> index f154465..1f6c243 100644
> --- a/server/sound.c
> +++ b/server/sound.c
> @@ -1508,7 +1508,7 @@ static void on_new_record_channel_client(SndChannel
> *channel, SndChannelClient *
>  {
>  spice_assert(client);
>  
> -channel->connection = client ;
> +channel->connection = client;
>  if (channel->volume.volume_nchannels) {
>  snd_set_command(client, SND_VOLUME_MASK);
>  }

Acked-by: Frediano Ziglio 

A bit picky I would say.

Frediano
___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Spice-devel] [spice-server 03/17] sound: Remove extraneous whitespace

2017-01-11 Thread Christophe Fergeau 
No need for this whitespace before ';'
---
 server/sound.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/sound.c b/server/sound.c
index f154465..1f6c243 100644
--- a/server/sound.c
+++ b/server/sound.c
@@ -1508,7 +1508,7 @@ static void on_new_record_channel_client(SndChannel 
*channel, SndChannelClient *
 {
 spice_assert(client);
 
-channel->connection = client ;
+channel->connection = client;
 if (channel->volume.volume_nchannels) {
 snd_set_command(client, SND_VOLUME_MASK);
 }
-- 
2.9.3

___
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel