jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject Re: [jclouds] JCLOUDS-1008: Add @Encoded annotation. (#861)
Date Tue, 20 Oct 2015 20:47:38 GMT
Ok. I really think it is necessary to isolate the encoding calls. We'll still have to process
the annotations in several places in the RestAnnotationProcessor, but the details on *how
to encode* (or how to call the encoding logic) shouldn't be spread along the class.
I agree to fix this in another PR (but let's make sure that happens :)) and will merge this
later today/early tomorrow.

Thanks for the great effort in these changes @timuralp!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/861#issuecomment-149697264
Mime
View raw message