nacx commented on this pull request. Thanks @justinThompson! This looks great. Thanks for taking your time to add the mock tests. Highly appreciated!
> +import com.google.common.collect.ImmutableMultimap.Builder; + +/** + * Binds the Iterable<String> to form parameters named with Subnets.member.index + */ +@Singleton +public class BindSubnetIdsToIndexedFormParams implements Binder { + + @SuppressWarnings("unchecked") + @Override + public <R extends HttpRequest> R bindToRequest(R request, Object input) { + Iterable<?> values = Iterable.class.cast(checkNotNull(input, "subnetIds")); + Builder<String, String> builder = ImmutableMultimap.builder(); + int i = 0; + for (Object o : values) { + builder.put("Subnets.member." + (i++ + 1), o.toString()); Initialize `i` to 1 to make the code easier to read? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs-aws/pull/70#pullrequestreview-18368668