> +   protected Function<org.jclouds.openstack.nova.v2_0.domain.SecurityGroup, 
> SecurityGroup> baseConverter;
> +   protected final Supplier<Map<String, Location>> locationIndex;
> +
> +   @Inject
> +   public 
> NovaSecurityGroupInZoneToSecurityGroup(Function<org.jclouds.openstack.nova.v2_0.domain.SecurityGroup,
>  SecurityGroup> baseConverter,
> +                                                 Supplier<Map<String, 
> Location>> locationIndex) {
> +      this.baseConverter = checkNotNull(baseConverter, "baseConverter");
> +      this.locationIndex = checkNotNull(locationIndex, "locationIndex");
> +   }
> +
> +   @Override
> +   public SecurityGroup apply(SecurityGroupInZone group) {
> +      SecurityGroupBuilder builder = 
> SecurityGroupBuilder.fromSecurityGroup(baseConverter.apply(group.getSecurityGroup()));
> +
> +      Location zone = locationIndex.get().get(group.getZone());
> +      checkState(zone != null, "location %s not in locationIndex: %s", 
> group.getZone(), locationIndex.get());

`checkArgument` or `checkNotNull`?

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

Reply via email to