> +import com.google.common.collect.ImmutableList.Builder;
> +import com.google.inject.Inject;
> +import org.jclouds.azurecompute.domain.DataVirtualHardDisk;
> +import org.jclouds.http.functions.ParseSax;
> +import org.xml.sax.Attributes;
> +
> +import java.util.List;
> +
> +public final class ListDataVirtualHardDisksHandler
> + extends
> ParseSax.HandlerForGeneratedRequestWithResult<List<DataVirtualHardDisk>> {
> + private boolean inDataVHD;
> + private final DataVirtualHardDiskHandler dataVirtualHardDiskHandler;
> + private final Builder<DataVirtualHardDisk> VHDs = ImmutableList.builder();
> +
> + @Inject
> + public ListDataVirtualHardDisksHandler(DataVirtualHardDiskHandler
> dataVirtualHardDiskHandler) {
In general, injection constructors should be package protected. Just remove the
public modifier:
```java
@Inject
ListDataVirtualHardDisksHandler(DataVirtualHardDiskHandler
dataVirtualHardDiskHandler) {
```
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/126/files#r23833496
