That might allow the close() to complete, but the underlying IOException should
not just be silently ignored. Since BufferedWriter.close() is allowed to throw
IOException, it should do so, after cleaning up the underlying Writer. I'm not
familiar with this code, but in most cases, judicious us
On 02/10/12 03:58 AM, Michael Barker wrote:
Hi Stuart,
Thank you for the reviewing the patches. I forgot about the
Contributed-by header, sorry about that. That's one I'll put on the
list to make sure I have sorted next time. I've split the
contributors up according to the two patches supplie
On 10/02/2012 13:16, Alex Lam S.L. wrote:
I think I have narrowed it down to this changeset:
http://hg.openjdk.java.net/hsx/hotspot-rt/jdk/rev/759aa847dcaf
7015589: (spec) BufferedWriter.close leaves stream open if close of
underlying Writer fails
Which no longer catches and ignores the except
I think I have narrowed it down to this changeset:
http://hg.openjdk.java.net/hsx/hotspot-rt/jdk/rev/759aa847dcaf
7015589: (spec) BufferedWriter.close leaves stream open if close of
underlying Writer fails
Which no longer catches and ignores the exception thrown by flush():
try (OutputStream os