Request for review: 7201156 : jar tool fails to convert file separation characters for list and extract
Alan Bateman
Alan.Bateman at oracle.com
Tue Nov 13 11:12:11 UTC 2012
On 13/11/2012 03:17, Sean Chou wrote:
> Hi Alan,
>
> Here is the updated webrev:
> http://cr.openjdk.java.net/~zhouyx/7201156/webrev.03/
> <http://cr.openjdk.java.net/%7Ezhouyx/7201156/webrev.03/> .
I think this looks much better.
One final comment, in createJarFile it looks like you forgot to remove
the close when you changed it to use try-with-resources. It's harmless,
just would be good to remove it before you push the change (no need to
generate a new webrev for this).
-Alan.
More information about the core-libs-dev
mailing list