JDK9 RFR of JDK-8029646: [pack200] should support the new zip64 format.
Alexander Zuev
alexander.zuev at oracle.com
Wed Jan 15 18:51:49 UTC 2014
On 1/15/14 21:34, Xueming Shen wrote:
> On 1/15/14 7:01 AM, Alexander Zuev wrote:
>> Hello,
>>
>> the new webrev with all the typos and comments fixed can be found
>> at http://cr.openjdk.java.net/~kizune/8029646/webrev.01/
>>
>> /Alex
>
> (1) jarmagic can be just a static constant somewhere or a stack
> variable. not big deal though.
Ok, i'll see to it.
> (2) the test only "tests" EOF for s. there is possibility that the
> newly created has more extra
> bytes at the end...though in theory this should not happen, it
> might be better just add an
> extra line to check the sizes of two first first?
The test tests for EOF only for s because if s is -1 (EOF) and d is not
(more extra bytes in newly created jar)
due to the check at line 99 - if (s != d) the exception will be thrown
for files content mismatch.
> (3) the rest of the change looks good, but I agreed with Kumar that
> you may need to add a
> regression test for a jar file with > 64k entries. otherwise
> the code for zip64 end is not
> being tested. the code looks fine, but I would trust a
> regression test more than my eyes:-)
All right, i'll modify the regression test.
/Alex
More information about the core-libs-dev
mailing list