Hi Lance, Looks good. I was wondering whether you might want to delete the files using jdk.test.lib.util.FileUtils.deleteFileIfExistsWithRetry() instead.
Brian > On Aug 6, 2019, at 1:31 PM, Lance Andersen <lance.ander...@oracle.com> wrote: > > Please review the fix for https://bugs.openjdk.java.net/browse/JDK-8226530 > <https://bugs.openjdk.java.net/browse/JDK-8226530> > <https://bugs.openjdk.java.net/browse/JDK-8226530 > <https://bugs.openjdk.java.net/browse/JDK-8226530>>, where ZipFile does not > return the correct size from the CEN when the Zip64 Extended Information > Extra Field is used to store the entry size > > The webrev can be found at > http://cr.openjdk.java.net/~lancea/8226530/webrev.00/index.html > <http://cr.openjdk.java.net/~lancea/8226530/webrev.00/index.html><http://cr.openjdk.java.net/~lancea/8226530/webrev.00/index.html > <http://cr.openjdk.java.net/~lancea/8226530/webrev.00/index.html>> > > Mach5 jdk tier1-tier3 all pass
