> +        }
> +
> +        public Builder offset(int offset){
> +            this.offset = offset;
> +            return this;
> +        }
> +
> +        public PaginationOptions build(){
> +            return new PaginationOptions(limit, offset, 0);
> +        }
> +    }
> +
> +    private int limit;
> +    private int offset;
> +    @Named("total_count")
> +    private final int totalCount;

Can we mark this field as final, and remove the "setters" in this class to make 
it immutable (and move the setter logic to the `build()` method)? The 
`PaginatedCollection` creates a new instance of the `PaginationOptions` when 
advancing, so there is no need to make it mutable.

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

Reply via email to