JDK9 RFR of JDK-8029646: [pack200] should support the new zip64 format.

Alexander Zuev alexander.zuev at oracle.com
Wed Jan 15 14:55:46 UTC 2014


Kumar,

   thanks for your findings. See my comments inline.

On 1/15/14 2:10, Kumar Srinivasan wrote:
> Hi Alex,
>
> zip.cpp: (nit) I would keep the hex values to be in upper case just 
> like the others for
>               consistency, not a big deal.
   Fixed.
>
>                typo: + // Zip64 END sugnature
   Also fixed.
>
>
> PackTestZip64.java: shouldn't we test a jar with 64K+  entries ?
   Do we really have to do so? Creating, repacking, packing, unpacking 
and comparing of such jar
file would take a lot of testing time. Right now i'm just testing that 
we are producing binary
identical jar after we normalized it. If you think creating of large jar 
file is necessary i can easily do so.

/Alex
>
> Kumar
>
>
>
> On 1/14/2014 10:04 AM, Alexander Zuev wrote:
>> Please review my fix for
>>  JDK-8029646: [pack200] should support the new zip64 format.
>>
>>   The fix can be found at 
>> http://cr.openjdk.java.net/~kizune/8029646/webrev.00/
>>
>>   Bug description is: https://bugs.openjdk.java.net/browse/JDK-8029646
>>
>> /Alex
>




More information about the core-libs-dev mailing list