> +      return this;
> +   }
> +
> +   /**
> +    * Sets the public ACL on the container so that anybody can read it.
> +    */
> +   public UpdateContainerOptions anybodyRead() {
> +      this.headers.put(CONTAINER_READ, CONTAINER_ACL_ANYBODY_READ);
> +      return this;
> +   }
> +
> +   /**
> +    * Sets the container that will contain object versions.
> +    */
> +   public UpdateContainerOptions versionsLocation(String containerName) {
> +      this.headers.put(CONTAINER_VERSIONS_LOCATION, containerName);

Would `VERSIONS_LOCATION_CONTAINER` make more sense here? I read 
"CONTAINER_VERSIONS_LOCATION" as "the versions location for the container", 
where (as far as I understand) it's actually "the container that is the 
location of the versions"?

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

Reply via email to