[Spice-devel] [PATCH spice-server 12/13] red-channel-client: Reduce indentation of some code

Victor Toso victortoso at redhat.com
Fri Aug 2 10:19:00 UTC 2019


Hi,

On Thu, May 30, 2019 at 03:22:53PM +0100, Frediano Ziglio wrote:
> Just a style change, return earlier to avoid some indentation.
> 
> Signed-off-by: Frediano Ziglio <fziglio at redhat.com>

Looks fine,
Acked-by: Victor Toso <victortoso at redhat.com>

> ---
>  server/red-channel-client.c | 37 +++++++++++++++----------------------
>  1 file changed, 15 insertions(+), 22 deletions(-)
> 
> diff --git a/server/red-channel-client.c b/server/red-channel-client.c
> index 4978f3567..3fd51d78b 100644
> --- a/server/red-channel-client.c
> +++ b/server/red-channel-client.c
> @@ -690,25 +690,21 @@ static void red_channel_client_ping_timer(void *opaque)
>      red_channel_client_cancel_ping_timer(rcc);
>  
>  #ifdef HAVE_LINUX_SOCKIOS_H /* SIOCOUTQ is a Linux only ioctl on sockets. */
> -    {
> -        int so_unsent_size = 0;
> +    int so_unsent_size = 0;
>  
> -        /* retrieving the occupied size of the socket's tcp snd buffer (unacked + unsent) */
> -        if (ioctl(rcc->priv->stream->socket, SIOCOUTQ, &so_unsent_size) == -1) {
> -            red_channel_warning(red_channel_client_get_channel(rcc),
> -                                "ioctl(SIOCOUTQ) failed, %s", strerror(errno));
> -        }
> -        if (so_unsent_size > 0) {
> -            /* tcp snd buffer is still occupied. rescheduling ping */
> -            red_channel_client_start_ping_timer(rcc, PING_TEST_IDLE_NET_TIMEOUT_MS);
> -        } else {
> -            red_channel_client_push_ping(rcc);
> -        }
> +    /* retrieving the occupied size of the socket's tcp snd buffer (unacked + unsent) */
> +    if (ioctl(rcc->priv->stream->socket, SIOCOUTQ, &so_unsent_size) == -1) {
> +        red_channel_warning(red_channel_client_get_channel(rcc),
> +                            "ioctl(SIOCOUTQ) failed, %s", strerror(errno));
>      }
> -#else /* ifdef HAVE_LINUX_SOCKIOS_H */
> +    if (so_unsent_size > 0) {
> +        /* tcp snd buffer is still occupied. rescheduling ping */
> +        red_channel_client_start_ping_timer(rcc, PING_TEST_IDLE_NET_TIMEOUT_MS);
> +        return;
> +    }
> +#endif /* ifdef HAVE_LINUX_SOCKIOS_H */
>      /* More portable alternative code path (less accurate but avoids bogus ioctls)*/
>      red_channel_client_push_ping(rcc);
> -#endif /* ifdef HAVE_LINUX_SOCKIOS_H */
>  }
>  
>  static inline int red_channel_client_waiting_for_ack(RedChannelClient *rcc)
> @@ -1140,16 +1136,13 @@ static int red_peer_receive(RedStream *stream, uint8_t *buf, uint32_t size)
>                  break;
>              } else if (errno == EINTR) {
>                  continue;
> -            } else if (errno == EPIPE) {
> -                return -1;
> -            } else {
> +            } else if (errno != EPIPE) {
>                  g_warning("%s", strerror(errno));
> -                return -1;
>              }
> -        } else {
> -            size -= now;
> -            pos += now;
> +            return -1;
>          }
> +        size -= now;
> +        pos += now;
>      }
>      return pos - buf;
>  }
> -- 
> 2.20.1
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20190802/28de419f/attachment.sig>


More information about the Spice-devel mailing list