> +import com.google.common.base.Strings;
> +
> +public abstract class BaseProfitBricksRequestBinder implements MapBinder {
> +
> +   protected final StringBuilder requestBuilder;
> +
> +   public BaseProfitBricksRequestBinder() {
> +      this.requestBuilder = new StringBuilder(255);
> +   }
> +
> +   protected String ifNotEmpty(String pattern, Object param) {
> +      return Strings.isNullOrEmpty((String) param) ? "" : 
> String.format(pattern, param);
> +   }
> +
> +   protected String mapOSType(OsType osType) {
> +      return osType == null ? "" : osType.value();

is value() different than toString() for these 2 (osType, zone)? If not, seems 
like you could just make one function as you are saying "empty string if object 
is null"

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

Reply via email to