> @@ -35,14 +36,25 @@ > // guice is single threaded. no need to lock this > List<Closeable> methodsToClose = Lists.<Closeable> newArrayList(); > > + volatile boolean isOpen;
`open` rather than `isOpen`? Matches bean naming conventions. And how about an AtomicBoolean rather than `volatile`? Then we don't need to synchronize the `close` method... --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/32/files#r4705028
