[PATCH FOR REVIEW] ResourceManager.getApplicationResources() does not close InputStreams (original) (raw)
Alan Bateman Alan.Bateman at oracle.com
Thu Nov 8 07:02:22 UTC 2012
- Previous message: [PATCH FOR REVIEW] ResourceManager.getApplicationResources() does not close InputStreams
- Next message: [PATCH FOR REVIEW] ResourceManager.getApplicationResources() does not close InputStreams
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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.
- Previous message: [PATCH FOR REVIEW] ResourceManager.getApplicationResources() does not close InputStreams
- Next message: [PATCH FOR REVIEW] ResourceManager.getApplicationResources() does not close InputStreams
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]