> + return this;
> + }
> + }
> +
> + /**
> + * Create and Update options - extend the domain class, passed to API
> update and create calls.
> + * Essentially the same as the domain class. Ensure validation and safe
> typing.
> + */
> + public static class CreateOptions extends Rule {
> + private CreateOptions(Rule rule) {
> + super(rule);
> + checkNotNull(rule.getDirection(), "direction should not be null");
> + checkNotNull(rule.getSecurityGroupId(), "security group id should
> not be null");
> + checkState(rule.getPortRangeMax()>= rule.getPortRangeMin(),
> + "port range max should be greater than or equal to port range
> min");
> + checkState(rule.getRemoteGroupId()==null ||
> rule.getRemoteIpPrefix()==null,
Again, `checkArg`? And spaces around the `==`?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/132/files#r16929817
