Hi Brian,

On 12/4/15 9:48 AM, Daniel Fuchs wrote:
> I wonder if:
> 248     public void close() throws IOException {
> 249          synchronized (lock) {
> 250               buf = null;
> 251          }
> 252          super.close();
> 253     }
>
> would be a safer implementation. I don't know the java.io code
> in depth - so that's just a question...

Actually scratch that. I see that all other super methods
are called from within the lock. So calling super.close()
from within the same lock as well is probably safer - and
what you have is good.

best regards,

-- daniel

Reply via email to