> +
> + return (inLowercase.startsWith(ACCOUNT_METADATA_PREFIX))
> + ? inLowercase : ACCOUNT_METADATA_PREFIX + inLowercase;
> + }
> +
> + @Override
> + public String toString() {
> + return "prefix: " + ACCOUNT_METADATA_PREFIX;
> + }
> + };
> + }
> +
> + @SuppressWarnings("unchecked")
> + @Override
> + public <R extends HttpRequest> R bindToRequest(R request, Object input) {
> + checkArgument(checkNotNull(input, "input") instanceof Map<?,?>, "this
> binder is only valid for Maps!");
I think this was actually meant to refer to the following comment:
> Remove one of the checks, since null instanceof X == false:
>
> checkArgument(input instanceof Map<?,?>, "input must be a non-null
> java.util.Map");
>
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/17/files#r6253996