[Spice-devel] [spice-gtk v1] Don't exceed one statement per line

[Marc-André Lureau]

ack

On Thu, Mar 1, 2018 at 10:23 AM, Victor Toso <victortoso at redhat.com> wrote:
> From: Victor Toso <me at victortoso.com>
>
> Cosmetic change. One break was dropped on _default:_ label of switch
>
> Signed-off-by: Victor Toso <victortoso at redhat.com>
> ---
>  src/channel-usbredir.c | 8 +++++---
>  src/spice-channel.c    | 7 +++++--
>  2 files changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/src/channel-usbredir.c b/src/channel-usbredir.c
> index a19df17..f509e49 100644
> --- a/src/channel-usbredir.c
> +++ b/src/channel-usbredir.c
> @@ -622,11 +622,13 @@ static void usbredir_log(void *user_data, int level, const char *msg)
>
>      switch (level) {
>          case usbredirparser_error:
> -            g_critical("%s", msg); break;
> +            g_critical("%s", msg);
> +            break;
>          case usbredirparser_warning:
> -            g_warning("%s", msg); break;
> +            g_warning("%s", msg);
> +            break;
>          default:
> -            CHANNEL_DEBUG(channel, "%s", msg); break;
> +            CHANNEL_DEBUG(channel, "%s", msg);
>      }
>  }
>
> diff --git a/src/spice-channel.c b/src/spice-channel.c
> index 1d66d05..c725903 100644
> --- a/src/spice-channel.c
> +++ b/src/spice-channel.c
> @@ -1355,8 +1355,11 @@ static void spice_channel_send_link(SpiceChannel *channel)
>
>      c->link_hdr.size = GUINT32_TO_LE(c->link_hdr.size);
>
> -    memcpy(p, &c->link_hdr, sizeof(c->link_hdr)); p += sizeof(c->link_hdr);
> -    memcpy(p, &link_msg, sizeof(link_msg)); p += sizeof(link_msg);
> +    memcpy(p, &c->link_hdr, sizeof(c->link_hdr));
> +    p += sizeof(c->link_hdr);
> +
> +    memcpy(p, &link_msg, sizeof(link_msg));
> +    p += sizeof(link_msg);
>
>      caps = SPICE_UNALIGNED_CAST(uint32_t *,p);
>      for (i = 0; i < c->common_caps->len; i++) {
> --
> 2.16.2
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



-- 
Marc-André Lureau