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