> @@ -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