Re: [spice-server v2 1/6] reds-stream: Introduce reds_stream_set_no_delay() helper

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

 



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

[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]