RFR: JDK-8232879: (zipfs) Writing out data with ZipFileSystem leads to a CRC failure in the generated jar file
Jaikiran Pai
jai.forums2013 at gmail.com
Sat Oct 26 02:10:00 UTC 2019
Thank you for the review, Lance.
On 26/10/19 4:25 AM, Lance Andersen wrote:
>
> The change to Zip FS looks good. I re-worked the test so that it also
> runs against ZipFile (which uses the CEN vs the LOC) and Zip FS in
> addition to ZipInputStream for extra coverage.
>
> The webrev can be found
> at: http://cr.openjdk.java.net/~lancea/8232879/webrev.00/index.html
Looks fine. A very minor note about
http://cr.openjdk.java.net/~lancea/8232879/webrev.00/test/jdk/jdk/nio/zipfs/CRCWriteTest.java.html
136 while ((ze = zis.getNextEntry()) != null) {
137 assertNotNull(ze);
Looks like an oversight - that assertNotNull(ze) on 137 isn't needed due to the != null check on 136.
-Jaikiran
More information about the core-libs-dev
mailing list