[Spice-devel] [PATCH spice-server] fixup! Convert RedChannelClient hierarchy to GObject

Christophe Fergeau cfergeau at redhat.com
Mon Oct 3 09:48:37 UTC 2016


On Fri, Sep 30, 2016 at 02:57:14PM +0100, Frediano Ziglio wrote:
> Split lines with more than 100 characters

> diff --git a/server/main-channel-client.h b/server/main-channel-client.h
> index f95ddef..360c61e 100644
> --- a/server/main-channel-client.h
> +++ b/server/main-channel-client.h
> @@ -212,7 +214,8 @@ red_channel_client_set_property(GObject *object,
>                  if (caps) {
>                      self->priv->remote_caps.num_caps = caps->len;
>                      free(self->priv->remote_caps.caps);
> -                    self->priv->remote_caps.caps = spice_memdup(caps->data, caps->len * sizeof(uint32_t));
> +                    self->priv->remote_caps.caps =
> +                        spice_memdup(caps->data, caps->len * sizeof(uint32_t));
>                  }
>              }
>              break;
> @@ -308,7 +311,8 @@ static void red_channel_client_class_init(RedChannelClientClass *klass)
>      g_object_class_install_property(object_class,
>                                      PROP_MONITOR_LATENCY,
>                                      g_param_spec_boolean("monitor-latency", "monitor-latency",
> -                                                         "Whether to monitor latency for this client",
> +                                                         "Whether to monitor latency "
> +                                                         "for this client",

I'd keep this one on a single line even if it's a bit more than 100 chars,
more readable this way imo.

>                                                           FALSE,
>                                                           G_PARAM_STATIC_STRINGS
>                                                           | G_PARAM_READWRITE |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20161003/150ce2c7/attachment.sig>


More information about the Spice-devel mailing list