nacx commented on this pull request.
> @@ -84,7 +84,7 @@ public static TagOptions keys(Set<String> keys) {
private void validateInput(final String input, int maxLength) {
checkNotNull(input);
checkState(input.length() <= maxLength, String.format("input must be <=
%d chars", maxLength));
- checkState(Iterables.any(FORBIDDEN_PREFIX, new Predicate<String>() {
+ checkState(Iterables.contains(FORBIDDEN_PREFIX, new Predicate<String>() {
Hmmmm If I understand this correctly, it is still not right.
You want to check that the given input **does not** contain any of the
forbidden inputs, right? It should be something like>
```java
checkState(!Iterables.any(FORBIDDEN_PREFIX, new Predicate<String>() {
```
--
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/pull/441#pullrequestreview-142055089
