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