> +import javax.inject.Singleton;
> +
> +import static com.google.common.base.Preconditions.checkArgument;
> +import static com.google.common.base.Preconditions.checkNotNull;
> +
> +/**
> + * @author Everett Toews
> + */
> +@Singleton
> +public class BindIdsToQueryParam implements Binder {
> +
> + @SuppressWarnings("unchecked")
> + @Override
> + public <R extends HttpRequest> R bindToRequest(R request, Object input) {
> + checkArgument(input instanceof Iterable<?>, "This binder is only valid
> for Iterable");
> + Iterable<String> ids = (Iterable<String>) checkNotNull(input,
> "Iterable of Strings");
[minor] I don't think you need the `checkNotNull` here, because `null` should
fail the `checkArgument` above
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/57/files#r7881649