On Fri, Mar 10, 2017 at 08:36:46AM -0500, Frediano Ziglio wrote: > > diff --git a/server/red-channel-client.c b/server/red-channel-client.c > > index a86833d..9fca8c1 100644 > > --- a/server/red-channel-client.c > > +++ b/server/red-channel-client.c > > @@ -571,13 +571,7 @@ static void > > red_channel_client_send_ping(RedChannelClient *rcc) > > } else { > > rcc->priv->latency_monitor.tcp_nodelay = delay_val; > > if (!delay_val) { > > - delay_val = 1; > > - if (setsockopt(rcc->priv->stream->socket, IPPROTO_TCP, > > TCP_NODELAY, &delay_val, > > - sizeof(delay_val)) == -1) { > > - if (errno != ENOTSUP) { > > - spice_warning("setsockopt failed, %s", > > strerror(errno)); > > - } > > - } > > + reds_stream_set_no_delay(rcc->priv->stream, TRUE); > > Why not true ? > (same below) Rest of the file still uses TRUE/FALSE, I think I prefer to keep things consistent for now. Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel