On 26/07/2019 17:41, Brian Burkhalter wrote:
Please see the updated patch which switches the implementation to the amended
version and incorporates the proposed change to the test:
http://cr.openjdk.java.net/~bpb/8078891/webrev.01/
That looks good to me Brian.
I had a look at the API documentation of AutoCloseable::close()
Though the further behavior of the closed object when close throws
an exception is a bit fuzzy - your amended implementation has greater
compatibility with preexisting behavior of SequenceInputStream, and
is in line with what one would expect by reading AutoCloseable::close().
Thanks Pavel for suggesting this!
best regards,
-- daniel
Thanks,
Brian
