Review Request: 8157391: jdeps left JarFile open

[Mandy Chung]

> On May 19, 2016, at 11:39 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> 
> 
> 
> On 20/05/2016 04:55, Mandy Chung wrote:
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8157391/webrev.00/index.html
>> 
>> tools/jdeps/modules/GenModuleInfo.java
>> tools/jdeps/modules/TransitiveDeps.java
>> tools/jdeps/modules/InverseDeps.java
>> 
>> These tests still failed on windows after JDK-8152502.
>> 
>> Now I have the jtreg logs showing that JAR files can’t be deleted although I still can’t reproduce them locally and on JPRT (will find out what’s the difference tomorrow).
>> 
> This looks okay to me. One thing you could do in the test is have JdepsUtil.Command be Closeable so that the tests can use try-with-resources and avoid failing with a file open.

Good point.  I’ll update that.

Mandy