JDK9 RFR of JDK-8029646: [pack200] should support the new zip64 format.
alexander.zuev at oracle.com
Wed Jan 15 14:55:46 UTC 2014
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.
> typo: + // Zip64 END sugnature
> 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.
> 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
>> Bug description is: https://bugs.openjdk.java.net/browse/JDK-8029646
More information about the core-libs-dev