Re: [spice-server v2 2/5] Remove unneeded red_channel_client_is_blocked() call

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

 



> 
> 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@xxxxxxxxxx>

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);
>          }
_______________________________________________
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]