This has been a nice cleanup and it is looking better now. We're almost there, 
thanks again!

A couple questions:

* Is there any reason why the fields in the domain classes are protected 
instead of private?
* Does this PR mean that the [existing 
CloudSigma](https://github.com/jclouds/jclouds/tree/master/apis/cloudsigma) api 
should be deprecated/removed?
* You managed to remove the custom response parsers (cool!), and I've seen that 
there are also many custom binders, used to map the domain objects to the 
request body. Since the domain objects are now following teh json structure and 
are properly annotated, would it be possible to use the existing 
`BindToJsonPayload` instead of the custom ones? For example:

```java
DriveInfo createDrive(@BinderParam(BindToJsonPayload.class) DriveInfo 
createDrive);
```

Sorry if I haven't seen this before, but this is quite an important amount of 
code and it is difficult to spot all things at once. You're doing a great job 
here, and I really don't want to make this a never-ending PR, so I hope these 
are the final major cleanups. I look forward to having this merged!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/12#issuecomment-22041977

Reply via email to