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