> +import java.io.Closeable;
> +import java.io.IOException;
> +
> +import org.jclouds.azurecompute.features.*;
> +
> +import javax.inject.Inject;
> +
> +public class AzureComputeApi implements Closeable {
> +    private final VirtualMachineApi virtualMachineApi;
> +    private final ImageApi imageApi;
> +    private final LocationApi locationApi;
> +    private final HostedServiceApi hostedServiceApi;
> +    private final RoleSizeApi roleSizeApi;
> +
> +    @Inject
> +    public AzureComputeApi(VirtualMachineApi virtualMachineApi, ImageApi 
> imageApi, LocationApi locationApi,

General: Following the Guice best practices, make all injection constructors 
package-private. This way only the injector (and the test classes) will be able 
to call it and you won't need to check for nulls, and other preconditions in 
the constructor code.

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

Reply via email to