On Jun 24, 2015, at 1:54 PM, Chris Hegarty <chris.hega...@oracle.com> wrote:

> The changes themselves look ok to me. Maybe the test could exercise 
> FilterOutputStream directly too?

I have updated the patch to exercise FilterOutputStream as well:

http://cr.openjdk.java.net/~bpb/8042377/webrev.01/

Barring objections to the contrary, I will plan to push it as-is tomorrow, 
Pacific Time.

On Jun 24, 2015, at 2:48 PM, Bernd Eckenfels <e...@zusammenkunft.net> wrote:

> I cannot speak about using AtomicBoolean in such basic classes, but I
> generally would have done the same as I think that concurrent close()
> is something which is not unlikely even for otherwise not thread safe
> streams.


Thanks for the corroboration.

Brian

Reply via email to