Hi Christoph, Thank you for doing this. Overall the changes look good.
A few minor thoughts ZIpFSTester.java - 537: Given you removed the method variable, I would tweak the printf string to make it a bit clearer that what the output its. JFSTester.java and MultiReleaseJarTest.java - I had the opposite feeling that having a jarfs subfolder adds clarity and moving MulltiReleaseJarTest.java might be the better way to go. It helps separate specific tests which are using the JarFileSystem from the ZipFileSystem especially as we add more, keeping things cleaner. Best Lance > On Apr 26, 2019, at 8:35 AM, Langer, Christoph <[email protected]> > wrote: > > Hi, > > please help reviewing these cleanups to the zipfs tests. In detail: > > - Fix warnings in test/jdk/jdk/nio/zipfs/Demo.java (not a real testcase but > Demo coding) > - change some calls of Runtime.version().major() to > Runtime.version().feature() (the former is deprecated) > - change occurences of new Integer() to Integer.valueOf() > - change occurences of class.newInstance() to > class.getDeclaredConstructor().newInstance() > - remove unused variables > - move test/jdk/jdk/nio/zipfs/jarfs/JFSTester.java to > test/jdk/jdk/nio/zipfs/JFSTester.java (not using the jarfs subfolder). There > is no need for this specific subfolder, given that MultiReleaseJarTest.java > neither isn't in the jarfs subfolder > > Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8223015.0/ > Bug: https://bugs.openjdk.java.net/browse/JDK-8223015 > > I verified that the tests still succeed with the change. > > Thanks > Christoph > <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 [email protected] <mailto:[email protected]>
