Re: [PATCH spice-server v2 04/12] red-channel-client: Remove only written field

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

 



On Tue, 2016-11-01 at 11:07 +0000, Frediano Ziglio wrote:
> id field of RedChannelClient is not used and marked as debugging.
> Also it's value is quite confusing.
> 
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx>
> ---
>  server/red-channel-client-private.h | 1 -
>  server/red-channel-client.c         | 1 -
>  2 files changed, 2 deletions(-)
> 
> diff --git a/server/red-channel-client-private.h b/server/red-
> channel-client-private.h
> index e77cdf9..38759ff 100644
> --- a/server/red-channel-client-private.h
> +++ b/server/red-channel-client-private.h
> @@ -87,7 +87,6 @@ struct RedChannelClientPrivate
>      } send_data;
>  
>      int during_send;
> -    int id; // debugging purposes
>      GQueue pipe;
>  
>      RedChannelCapabilities remote_caps;
> diff --git a/server/red-channel-client.c b/server/red-channel-
> client.c
> index 42f1d2c..a28f5e6 100644
> --- a/server/red-channel-client.c
> +++ b/server/red-channel-client.c
> @@ -924,7 +924,6 @@ static gboolean
> red_channel_client_initable_init(GInitable *initable,
>          self->priv->latency_monitor.roundtrip = -1;
>      }
>  
> -    self->priv->id = red_channel_get_n_clients(self->priv-
> >channel);
>      red_channel_add_client(self->priv->channel, self);
>      red_client_add_channel(self->priv->client, self);
>  
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://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]