> +
> + protected CreateBuilder self() {
> + 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(),
Should this be `checkArgument` rather than `checkState`? Also, space before the
`>=`?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/132/files#r16929813
