> @@ -35,14 +35,32 @@
>     // guice is single threaded. no need to lock this
>     List<Closeable> methodsToClose = Lists.<Closeable> newArrayList();
>  
> +   public enum State {
> +       OPEN,
> +       CLOSING,
> +       CLOSED
> +   }
> +
> +   State state;

We're now in a possible race condition here. Is there any reason this field 
isn't private, by the way? Do we need an AtomicReference here?

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

Reply via email to