RFR - 8132734: java.util.jar.* changes to support multi-release jar files
Alan Bateman
Alan.Bateman at oracle.com
Sun Nov 15 12:51:31 UTC 2015
On 11/11/2015 16:44, Steve Drach wrote:
> Hi,
>
> Please review the new webrev that addresses the issues raised by
> Sherman and Alan in the last iteration. In particular:
> - fixed the race condition in isMultiRelease() and another one with
> the variables ‘version’ and ‘configured’
> - changed the fragment for JarURLConnection runtime versioning from
> ‘rtversioned’ to ‘runtime’, and created documentation for it
> - used try with resources to open JarFile in all the tests
>
> Issue: https://bugs.openjdk.java.net/browse/JDK-8132734
> JEP 238: https://bugs.openjdk.java.net/browse/JDK-8047305
> Webrev:
> http://cr.openjdk.java.net/~psandoz/multiversion-jar/jar-webrev/
> <http://cr.openjdk.java.net/%7Epsandoz/multiversion-jar/jar-webrev/>
>
The updated webrev looks must better. In JarURLConnection then it would
be good if the reference to multi-release JARs should link to the
description in the JarFile spec.
In the previous round then we were discussing renaming the
jdk.util.jar.multirelease property. Has there been any more on that?
The test MultiReleaseJarURLConnection uses @library
/lib/testlibrary/java/util/jar so it's reaching across the file system
to use the infrastructure of the JarFile tests. It might be clearer to
move the test to the JarFile directory.
It would be nice if we could reduce some of the really long lines if
possible, just to make future side-by-side a bit easier (avoid
horizontal scrolling).
-Alan.
More information about the core-libs-dev
mailing list