> +      public Builder name(String name) {
> +         this.name = name;
> +         return this;
> +      }
> +
> +      /**
> +       * @param metadata The metadata of this Webhook.
> +       * @return The builder object.
> +       * @see Webhook#getMetadata()
> +       */
> +      public Builder metadata(Map<String, Object> metadata) {
> +         if (metadata == null) {
> +            this.metadata = ImmutableMap.of();
> +         } else {
> +            this.metadata = ImmutableMap.copyOf(metadata);
> +         }

The constructor already does this logic. Could this just be `this.metadata = 
metadata`?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/48/files#r7512803

Reply via email to