RFR: JDK-8073158 zip files with total entry count 0xFFFF need not be ZIP64 files
Xueming Shen
xueming.shen at oracle.com
Wed Mar 25 17:55:40 UTC 2015
It looks fine...I hope you guys also have some tests over there to bring
in more confidence :-)
It might be "easier" to simply update the original haveZIP64() with the code
we have in zip_util.c in which we also try to read the end64 to verify if we
really have one. Your choice though.
-Sherman
On 03/25/2015 09:55 AM, Martin Buchholz wrote:
> Yeah, this review is kinda scary. There's a lot of technical debt here, and this change only addresses some of it.
>
> A variant of this code is in use at Google.
>
> On Mon, Mar 23, 2015 at 1:18 PM, Martin Buchholz <martinrb at google.com <mailto:martinrb at google.com>> wrote:
>
> Hi Xueming and Alan,
>
> I'd like you to do a code review.
>
> https://bugs.openjdk.java.net/browse/JDK-8073158
> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/0xffff-entries-zip-file/ <http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/0xffff-entries-zip-file/>
>
> Of course, the really correct thing is to have at most one zip implementation per programming language, but I'm not trying to fix that here (how many zip implementations does openjdk have?)
>
>
More information about the core-libs-dev
mailing list