You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jclouds.apache.org by Andrew Phillips <no...@github.com> on 2013/11/01 01:26:25 UTC

Re: [jclouds-labs-openstack] JCLOUDS-215 - Group config, Group launch, Scaling policy functionality (#39)

>        this.activeCapacity = activeCapacity;
>        this.pendingCapacity = pendingCapacity;
>        this.desiredCapacity = desiredCapacity;
>        this.paused = paused;
> -      this.groupInstances = ImmutableList.copyOf(groupInstances);
> +      if(groupInstances == null) {
> +         this.groupInstances = ImmutableList.of();
> +      } else {
> +         this.groupInstances = ImmutableList.copyOf(groupInstances);
> +      }

[minor] factory out into a "copyOfOrEmpty" or similar method. If not, add a space in `if(...`?

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