[jclouds/jclouds] Remove Inject on constructors for abstract classes (#1143)

[Andrew Gaul <no...@github.com>]

This is not meaningful since these classes cannot be instantiated.
Found via error-prone.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1143

-- Commit Summary --

  * Remove Inject on constructors for abstract classes

-- File Changes --

    M apis/ec2/src/main/java/org/jclouds/ec2/xml/BaseReservationHandler.java (5)
    M blobstore/src/main/java/org/jclouds/blobstore/internal/BaseBlobStore.java (2)
    M core/src/main/java/org/jclouds/http/internal/BaseHttpCommandExecutorService.java (4)
    M providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/xml/BaseAWSReservationHandler.java (4)
    M providers/digitalocean2/src/main/java/org/jclouds/digitalocean2/functions/BaseToPagedIterable.java (4)
    M providers/packet/src/main/java/org/jclouds/packet/functions/BaseToPagedIterable.java (4)
    M providers/profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/firewall/BaseFirewallResponseHandler.java (5)
    M providers/profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/firewall/rule/BaseFirewallRuleResponseHandler.java (5)
    M providers/profitbricks/src/main/java/org/jclouds/profitbricks/http/parser/storage/BaseStorageResponseHandler.java (5)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1143.patch
https://github.com/jclouds/jclouds/pull/1143.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1143

Re: [jclouds/jclouds] Remove Inject on constructors for abstract classes (#1143)

[Ignasi Barrera <no...@github.com>]

nacx approved this pull request.





-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1143#pullrequestreview-61412457