RFR: 8304013: Add a fast, non-manual alternative to test/jdk/java/util/zip/ZipFile/TestTooManyEntries [v3]
Lance Andersen
lancea at openjdk.org
Sat Mar 25 11:04:21 UTC 2023
On Sat, 25 Mar 2023 06:46:22 GMT, Eirik Bjorsnos <duke at openjdk.org> wrote:
>> test/jdk/java/util/zip/ZipFile/EndOfCenValidation.java line 120:
>>
>>> 118: /**
>>> 119: * Validate that an end of central directory record with a
>>> 120: * CEN size which exceeds the position of the EOC record is rejected.
>>
>> EOC is not a standard zip file technical term. Did you mean "END header" ?
>
> If by "standard" you mean "APPNOTE.TXT", then that uses "end of central directory record" which is a bit long.
>
> The Java implementation seems to use END, like in `ZipFile.ENDHDR`, "END header", etc.
>
> It is probably better to be consistent than correct here, so I've changed the test to use "END header" consistently when referring to the "end of central directory record".
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 then the ZipFile constant verbiage, but that is just me :-)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12231#discussion_r1148350853
More information about the core-libs-dev
mailing list