> +import org.jclouds.profitbricks.domain.server.AvailabilityZone;
> +import org.jclouds.rest.MapBinder;
> +
> +public abstract class BaseProfitBricksRequestBinder implements MapBinder {
> +
> + protected final StringBuilder requestBuilder;
> +
> + public BaseProfitBricksRequestBinder() {
> + this.requestBuilder = new StringBuilder(32);
> + }
> +
> + protected String ifNotEmpty(String pattern, Object param) {
> + return Strings.isNullOrEmpty((String) param) ? "" :
> String.format(pattern, param);
> + }
> +
> + protected String rightAway(String pattern, Object param) {
The intent of this method is ambiguous. I would suggest eliminating this method
in favor of `String.format()`.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/72/files#r18046303
