> > On Mon, Oct 09, 2017 at 08:08:24AM -0400, Frediano Ziglio wrote: > > > @@ -1671,6 +1675,10 @@ static void > > > red_channel_client_pipe_clear(RedChannelClient *rcc) > > > > > > void red_channel_client_ack_zero_messages_window(RedChannelClient *rcc) > > > { > > > + if (red_channel_client_waiting_for_ack(rcc)) { > > > + red_channel_client_watch_update_mask(rcc, > > > + > > > SPICE_WATCH_EVENT_READ|SPICE_WATCH_EVENT_WRITE); > > > + } > > > rcc->priv->ack_data.messages_window = 0; > > > } > > > > > > > Looks good. I would remove the if condition. > > Considered both options, I'll remove it then ;) > > Christophe > Ack :-) Frediano _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel