Re: [PATCH spice-server] fixup! Convert RedChannelClient hierarchy to GObject

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

 



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 |

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]