> @@ -65,8 +65,7 @@
>           timeoutMillis = timeoutMillis.or(defaultMillis);
>        } else {
>           // TODO: remove old logic once Named annotations are on all methods

Sadly, all methods don't currently contain Named. This is one of the
reasons I suggest we use error-prone to help us be able to remove
statements like this more authoritatively. For example, we could make an
error-prone bug pattern for forgetting to add the Named annotation which
will make the build fail.

https://code.google.com/p/error-prone/wiki/BugPatterns

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

Reply via email to