> +            return "";
> +         }
> +      };
> +   }
> +
> +   @SuppressWarnings("unchecked")
> +   @Override
> +   public <R extends HttpRequest> R bindToRequest(R request, Object input) {
> +      checkArgument(input instanceof Map<?,?>, "this binder is only valid 
> for Maps!");
> +      checkNotNull(request, "request");
> +
> +      // Transform the map to filter only the prefixed values
> +      Map<String, String> userMetadata = Maps2.transformKeys((Map<String, 
> String>) input, transformHeaders);
> +
> +      // Transform the values to empty 
> +      Map<String, String> deletedMetadata = 
> Maps.transformValues(userMetadata, transformValues);

Do we really think this is easier to read and understand than:
```
// Transform the values to empty
for (String key : userMetadata.keySet()) {
  userMetadata.put(key, "");
}
```
?

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

Reply via email to