[Spice-devel] [spice-server v2 2/5] Remove unneeded red_channel_client_is_blocked() call

Frediano Ziglio fziglio at redhat.com
Fri Apr 28 12:02:53 UTC 2017


> 
> red_channel_client_msg_sent() calls red_channel_client_clear_sent_item()
> which will clear the rcc->priv->send_data.blocked flag. Later on, it
> contains a check for !red_channel_client_is_blocked(), which will always
> be true as nothing in between could have set it again. This commit
> removes this unneeded check.
> 
> This check was already redundant when it was introduced in
> 9a62a9a809eaf0
> 
> Signed-off-by: Christophe Fergeau <cfergeau at redhat.com>

Acked

Frediano

> ---
>  server/red-channel-client.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/server/red-channel-client.c b/server/red-channel-client.c
> index f9054eaa9..1c4b27df0 100644
> --- a/server/red-channel-client.c
> +++ b/server/red-channel-client.c
> @@ -652,8 +652,7 @@ static void red_channel_client_msg_sent(RedChannelClient
> *rcc)
>          spice_assert(rcc->priv->send_data.header.data != NULL);
>          red_channel_client_begin_send_message(rcc);
>      } else {
> -        if (!red_channel_client_is_blocked(rcc)
> -            && g_queue_is_empty(&rcc->priv->pipe)) {
> +        if (g_queue_is_empty(&rcc->priv->pipe)) {
>              /* It is possible that the socket will become idle, so we may be
>              able to test latency */
>              red_channel_client_restart_ping_timer(rcc);
>          }


More information about the Spice-devel mailing list