Code review request 7077769: (zipfs) ZipFileSystem.writeCEN() writes wrong "data size" for ZIP64 extended information extra field
Xueming Shen
xueming.shen at oracle.com
Fri Aug 26 21:29:11 UTC 2011
On 08/26/2011 01:17 PM, Alan Bateman wrote:
> Xueming Shen wrote:
>> Alan
>>
>> Webrev has been updated to
>>
>> (1) use the new try(resource){}
>> (2) update the existing test case LargetZip to
>> a) be able to add one more entry at the end of the > 4g zip via
>> ZipFileSystem
>> b) read through all entries inside the large zipfile, given
>> zipfile's self-crc-check
>> it guarantees the correctness of the zipfile. So it now goes
>> through 18G+
>> file read/write for one round:-)
>> (you have to use -XX:-UseLoopPredicate on server vm for now to
>> workaround
>> the C2 loop redicate bug)
>>
>> http://cr.openjdk.java.net/~sherman/7077769/webrev/
> Looks good. On the test I guess I would have used Files.walkFileTree
> in the update method. Also in the mkdirs implementation can be
> replaced with Files.createDirectories.
Webrev has been updated to use Files.walkFileTree and
Files.createDirectories.
Thanks,
Sherman
>
> -Alan.
More information about the core-libs-dev
mailing list