RFR: 8304013: Add a fast, non-manual alternative to test/jdk/java/util/zip/ZipFile/TestTooManyEntries
Lance Andersen
lancea at openjdk.org
Fri Mar 24 12:29:32 UTC 2023
On Fri, 24 Mar 2023 08:05:19 GMT, Eirik Bjorsnos <duke at openjdk.org> wrote:
>> test/jdk/java/util/zip/ZipFile/EndOfCenValidation.java line 177:
>>
>>> 175: Path zip) throws IOException {
>>> 176:
>>> 177: Files.deleteIfExists(zip);
>>
>> Is this needed? I think I would just call the cleanup method from the setup method
>
> Sparse files need to be created as such. The purpose of this delete exists to make this explicit. I moved this closer to the FileChannel.open and added a comment to explain the delete.
I am fine leaving it, but still not convinced it is needed if the setup method calls the cleanup method and should achieve the same result. Again no big deal.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/12231#discussion_r1147515419
More information about the core-libs-dev
mailing list