[Spice-devel] [PATCH v9 10/11] sound: Make clear active and client_active are boolean

[Jonathon Jongsma]

Acked-by: Jonathon Jongsma <jjongsma at redhat.com>


On Tue, 2016-12-20 at 17:44 +0000, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
> ---
>  server/sound.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/server/sound.c b/server/sound.c
> index 33d0e96..351cd8d 100644
> --- a/server/sound.c
> +++ b/server/sound.c
> @@ -93,8 +93,8 @@ GType snd_channel_client_get_type(void)
> G_GNUC_CONST;
>  struct SndChannelClient {
>      RedChannelClient parent;
>  
> -    int active;
> -    int client_active;
> +    gboolean active;
> +    gboolean client_active;
>  
>      uint32_t command;
>  
> @@ -176,7 +176,7 @@ struct SndChannel {
>      SndChannelClient *connection; /* Only one client is supported */
>      SndChannel *next; /* For the global SndChannel list */
>  
> -    int active;
> +    gboolean active;
>      SpiceVolumeState volume;
>      uint32_t frequency;
>  };
> @@ -860,7 +860,7 @@ static void snd_playback_start(SndChannel
> *channel)
>  {
>      SndChannelClient *client = channel->connection;
>  
> -    channel->active = 1;
> +    channel->active = TRUE;
>      if (!client)
>          return;
>      spice_assert(!client->active);
> @@ -883,7 +883,7 @@ SPICE_GNUC_VISIBLE void
> spice_server_playback_stop(SpicePlaybackInstance *sin)
>  {
>      SndChannelClient *client = sin->st->channel.connection;
>  
> -    sin->st->channel.active = 0;
> +    sin->st->channel.active = FALSE;
>      if (!client)
>          return;
>      PlaybackChannelClient *playback_client =
> PLAYBACK_CHANNEL_CLIENT(client);
> @@ -1153,7 +1153,7 @@ static void snd_record_start(SndChannel
> *channel)
>  {
>      SndChannelClient *client = channel->connection;
>  
> -    channel->active = 1;
> +    channel->active = TRUE;
>      if (!client) {
>          return;
>      }
> @@ -1179,7 +1179,7 @@ SPICE_GNUC_VISIBLE void
> spice_server_record_stop(SpiceRecordInstance *sin)
>  {
>      SndChannelClient *client = sin->st->channel.connection;
>  
> -    sin->st->channel.active = 0;
> +    sin->st->channel.active = FALSE;
>      if (!client)
>          return;
>      spice_assert(client->active);