Re: [Spice-devel] [PATCH spice-server 03/10] reds-stream: Remove shutdown field

2017-09-11 Thread Christophe Fergeau
On Mon, Sep 11, 2017 at 11:56:45AM -0400, Frediano Ziglio wrote: > > > > > > Acked-by: Christophe Fergeau > > > > On Mon, Sep 11, 2017 at 11:15:40AM +0100, Frediano Ziglio wrote: > > > This field was used only by RedChannelClient to mark when the > > > socket was shutdown.

Re: [Spice-devel] [PATCH spice-server 03/10] reds-stream: Remove shutdown field

2017-09-11 Thread Frediano Ziglio
> > > Acked-by: Christophe Fergeau > > On Mon, Sep 11, 2017 at 11:15:40AM +0100, Frediano Ziglio wrote: > > This field was used only by RedChannelClient to mark when the > > socket was shutdown. This condition can simply be tested by > > RedChannelClient checking if

Re: [Spice-devel] [PATCH spice-server 03/10] reds-stream: Remove shutdown field

2017-09-11 Thread Christophe Fergeau
Acked-by: Christophe Fergeau On Mon, Sep 11, 2017 at 11:15:40AM +0100, Frediano Ziglio wrote: > This field was used only by RedChannelClient to mark when the > socket was shutdown. This condition can simply be tested by > RedChannelClient checking if there's a watch as is

[Spice-devel] [PATCH spice-server 03/10] reds-stream: Remove shutdown field

2017-09-11 Thread Frediano Ziglio
This field was used only by RedChannelClient to mark when the socket was shutdown. This condition can simply be tested by RedChannelClient checking if there's a watch as is the only condition (beside object destroying) where the watch is removed. In any case the shutdown was used to understand if