>     public void addToClose(Closeable toClose) {
>        methodsToClose.add(toClose);
>     }
>  
> -   public void close() throws IOException {
> +    public synchronized void close() throws IOException {
> +      this.isOpen = false;

One consequence of this is that `isOpen` will be false even **before** all the 
Closeables have been closed. I assume this is by design? In any case, it's not 
covered by a test case.

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

Reply via email to