On Mon, Feb 23, 2015 at 10:09:49AM -0500, Marc-André Lureau wrote: > Hi > > ----- Original Message ----- > > > Because that field is for peer closing. I can rename it peer_closed > > > perhaps? (peer->peer_closed...) > > > > You lost me here. Do you mean PipeInputStream::closed means that the > > peer PipeOutputStream has been closed? If that's what you mean, then it > > definitely should be peer_closed. > > I don't mind, for me the field means the pipe is closing, either side. > However, since it's internal to the pipe implementation and is only > set by the peer, I guess it could be closed peer_closed too. That > really doesn't change much of the meaning of this code anyway. pipe_closed then if that variable does not specifically refer to either side of the pipe being closed ? Christophe
Attachment:
pgpGfMJluIEEq.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel