> +            .add("portRangeMax", portRangeMax)
> +            .add("protocol", protocol)
> +            .add("remoteGroupId", remoteGroupId)
> +            .add("remoteIpPrefix", remoteIpPrefix)
> +            .toString();
> +   }
> +
> +
> +   /*
> +    * Methods to get the Create and Update builders follow
> +    */
> +
> +   /**
> +    * @return the Builder for creating a new SecurityGroupRule
> +    */
> +   public static CreateBuilder createOptions(RuleDirection direction, String 
> securityGroupId) {

> It's a bit confusing. createOptions returns a builder that allows you to 
> create a CreateOptions object.

Thanks for explaining - became clearer as I saw more of the code ;-) I'm still 
uneasy about the use of `*Options` as a name for a domain object class, even 
though with the intended usage pattern I guess it's meant to be hidden.

I just sent an email to the dev list - hopefully someone will have some bright 
suggestion...

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

Reply via email to