> +   private final ReturnStringIf2xx returnStringIf200;
> +   private final ListOfKeyValuesDelimitedByBlankLinesToListOfMaps 
> mapConverter;
> +   private final MapToStandardDrive mapToStandardDrive;
> +
> +   @Inject
> +   ListOfKeyValuesDelimitedByBlankLinesToStandardDriveSet(ReturnStringIf2xx 
> returnStringIf200,
> +         ListOfKeyValuesDelimitedByBlankLinesToListOfMaps mapConverter, 
> MapToStandardDrive mapToStandardDrive) {
> +      this.returnStringIf200 = checkNotNull(returnStringIf200, 
> "returnStringIf200");
> +      this.mapConverter = checkNotNull(mapConverter, "mapConverter");
> +      this.mapToStandardDrive = checkNotNull(mapToStandardDrive, 
> "mapToStandardDrive");
> +   }
> +
> +   @Override
> +   public Set<StandardDrive> apply(HttpResponse response) {
> +      String text = returnStringIf200.apply(response);
> +      if (text == null || text.trim().equals(""))

Just out of curiosity, when would this be expected to return a non-empty string 
of spaces? And use `{...}` after the if?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/362/files#r13033913

Reply via email to