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

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

 



On Mon, Sep 11, 2017 at 11:56:45AM -0400, Frediano Ziglio wrote:
> > 
> > 
> > Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> > 
> > 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 the only
> > > condition (beside object destroying) where the watch is removed.
> > 
> > It's removed on red_channel_client_disconnect(), dunno if that's what
> > you call "object destroying"
> > 
> 
> After a disconnection the object should be destroyed.
> 
> I could update to
> 
> ...condition (beside object destroying/disconnecting) where the watch is removed.

Fine with me, thanks.

Christophe

> 
> Frediano
_______________________________________________
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]