Modular JAR file support
Alan Bateman
Alan.Bateman at oracle.com
Wed Apr 6 11:08:43 PDT 2011
Mandy Chung wrote:
>
> :
>
> Webrev at:
> http://cr.openjdk.java.net/~mchung/jigsaw/webrevs/modular-jar-file/
I went through the webrev and the changes mostly look good to me, easy
to follow. A general comment is that the install is open to races with
other concurrent access to the module library but that's a bigger
problem, not a big concern for this.
I read the mail from David Schlosnagle on the missing close and a
related suggestion for using try-with-resources is that
ModuleFileFormat.Reader could implement Closeable which would allow a
bit of clean-up in SimpleLibrary.install.
Minor nit, the copyright year on the test in 2009.
-Alan.
More information about the jigsaw-dev
mailing list