Not a reviewer, but looks good to me. :-)
Kind regards,
Jonathan
On 12 Oct 2016 23:54, "Lance Andersen" wrote:
> +1
> > On Oct 12, 2016, at 6:52 PM, Mandy Chung wrote:
> >
> > Simple patch close the ClassFileReader with try-with-resource.
> >
> >
> > diff --git a/src/jdk.jdeps/share/classes/co
+1
> On Oct 12, 2016, at 6:52 PM, Mandy Chung wrote:
>
> Simple patch close the ClassFileReader with try-with-resource.
>
>
> diff --git a/src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java
> b/src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java
> --- a/src/jdk.jdeps/sh
Simple patch close the ClassFileReader with try-with-resource.
diff --git a/src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java
b/src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java
--- a/src/jdk.jdeps/share/classes/com/sun/tools/jdeps/JdepsTask.java
+++ b/src/jdk.jdeps/sh