On 07/11/2012 20:51, Andrew Hughes wrote:
:
As you can see on the IcedTea bug, I've asked the same question.
I'd have preferred it to use try-with-resources myself (easier to
follow for one thing), but given the patch is as it is, I'm now
wary about changing it and negating the existing testing that's
already been done.
I see you've pushed this already but I think we need to open a follow-on
bug on this as it looks like there is additional work to do.
Particularly the case where there are several streams and close fails
(looks to me that it will not attempt to close all streams in that
case). Also, given that the target is jdk8 then it seems reasonable to
see if try-with-resources could be used for the second case, I assume
the testing you mentioned was with a different release. The other thing
is that we try to include a regression test with all fixes where
possible, I don't know how feasible it for this case.
-Alan.
