> On 12 Sep 2016, at 08:10, Claes Redestad <claes.redes...@oracle.com> wrote: > > 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/ >
+1 (post review i see it is already committed). Our optimisation to compress the “optoSft” data was premature, i guess we had an error in the initial data. Paul.