> +      return builder()
> +            .name(name)
> +            .template(template)
> +            .templateUrl(templateUrl)
> +            .parameters(parameters)
> +            .disableRollback(disableRollback)
> +            .files(files)
> +            .environment(environment).build();
> +   }
> +
> +   public static final class Builder {
> +
> +      private String name;
> +      private String template;
> +      private String templateUrl;
> +      private Map<String, Object> parameters = ImmutableMap.of();

Instead of re-assigning the map if someone sets it, you could better use an 
ImmutableMap.Builder here.

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

Reply via email to