RFR: 8304013: Add a fast, non-manual alternative to test/jdk/java/util/zip/ZipFile/TestTooManyEntries [v4]

Eirik Bjorsnos duke at openjdk.org
Sun Mar 26 17:01:41 UTC 2023


On Sun, 26 Mar 2023 16:51:28 GMT, Martin Buchholz <martin at openjdk.org> wrote:

>> So, what I would suggest is adding a comment somewhere regarding _end of central directory record_ and note the reference to _ENDHDR_ or _END Header_
>> 
>> I don't think you want to spend too much time here just the wording reference the _end of central directory record_ was clear and where you were abbreviating,  then I would reference ENDHDR
>> 
>> I would be looking at the APP.NOTE  when looking at this test, not the ZipFile constant verbiage, but that is just me :-)
>
> Neither APPNOTE nor JDK doc zip  terminology is great, but a test is not the place to invent something better.
> 
> (But I'm surprised no one seems to be using "EOCD")

I think that would be “EOCDR”. (Hides behind the bike shed..)

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/12231#discussion_r1148595939


More information about the core-libs-dev mailing list