> +import static com.google.common.base.Preconditions.checkArgument;
> +import static com.google.common.base.Preconditions.checkNotNull;
> +import static com.google.common.base.Preconditions.checkState;
> +
> +import java.net.URI;
> +import java.util.Map;
> +
> +import javax.inject.Inject;
> +import javax.inject.Singleton;
> +
> +import org.jclouds.location.Region;
> +
> +import com.google.common.base.Function;
> +import com.google.common.base.Supplier;
> +
> +@Singleton

fyi: we over-use singleton, and I'd recommend removing it. Singleton matters 
when you have shared state that wasn't given to you in the constructor.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/169/files#r20374105

Reply via email to