RFR: 8165723: JarFile::isMultiRelease() method returns false when it should return true
Claes Redestad
claes.redestad at oracle.com
Mon Sep 12 15:10:15 UTC 2016
On 2016-09-12 16:24, Alan Bateman wrote:
> This looks okay.
Thanks for the review!
> For the MultiReleaseJarAPI test then you probably should use
> jdk.testlibrary.RandomFactory so that the seed is recorded in the
> output for when the test fails.
Done.
> Also in CreateMultiReleaseTestJars then it might be cleaner to have a
> separate method to add extra stuff to the JAR file - I think that
> would make the usages a bit easier to read.
I took a second look, realized
CreateMultiReleaseTestJars::buildShortMultiReleaseJar is no longer being
used by any test, and simplified things a bit accordingly:
http://cr.openjdk.java.net/~redestad/8165723/webrev.02/
Thanks!
/Claes
More information about the core-libs-dev
mailing list