> +   @Override
> +   public boolean equals(Object obj) {
> +      if (obj == null)
> +         return false;
> +      if (getClass() != obj.getClass())
> +         return false;
> +      ContentRange other = (ContentRange) obj;
> +      return Objects.equal(this.from, other.to) && Objects.equal(this.from, 
> other.to);
> +   }
> +
> +   @Override
> +   public String toString() {
> +      return from + "-" + to;
> +   }
> +
> +   public static final class Builder {

What benefit does `Builder` give in this case?  All of its methods are single 
factories which return a `ContentRange`.  Do these belong as part of the outer 
class instead of the inner class?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/18/files#r13990114

Reply via email to