> +
> +public abstract class BaseProfitBricksRequestBinder<T> implements MapBinder {
> +
> + protected final StringBuilder requestBuilder;
> + protected String paramName;
> +
> + public BaseProfitBricksRequestBinder() {
> + this.requestBuilder = new StringBuilder(255);
> + }
> +
> + @Override
> + public <R extends HttpRequest> R bindToRequest(R request, Map<String,
> Object> postParams) {
> + checkNotNull(request, "request");
> +
> + checkNotNull(paramName, "Initialize 'paramName' in constructor");
> + Object obj = checkNotNull(postParams.get(paramName), format("Param
> '%s' cannot be null.", paramName));
Minor: there's no need to manually format the message. The
[checkNotNull(T,String,Object...)](http://docs.guava-libraries.googlecode.com/git-history/release/javadoc/com/google/common/base/Preconditions.html#checkNotNull(T,%20java.lang.String,%20java.lang.Object...))
method already does that.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/72/files#r22133395
