> +         return this;
> +      }
> +
> +      /**
> +       * @see IpPermission#getPorts()
> +       */
> +      public Builder addPort(Integer port) {
> +         this.ports.add(singleton(checkNotNull(port, "port")));
> +         return this;
> +      }
> +
> +      /**
> +       * @see IpPermission#getPorts()
> +       */
> +      public Builder addPortRange(Integer start, Integer end) {
> +         checkState(checkNotNull(start, "start") < checkNotNull(end, "end"),

`checkArgument`? This doesn't seem like a case for `IllegalStateException`.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/145/files#r6228651

Reply via email to