> @@ -33,8 +33,8 @@
> @Singleton
> public class BlockIdValidator extends Validator<String> {
> @Override
> - public void validate(@Nullable String s) throws IllegalArgumentException {
> - if (s.length() > 64)
> + public void validate(String s) throws IllegalArgumentException {
I removed it because; technically it can't; it would immediately throw an
IllegalArgumentException. Null isn't valid. The only reason it was in there was
because the interface has that on there; but I don't think that's even the
correct annotation for the interface in most circumstances.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/66/files#r5156293
