ZipOutputStream & directories
Alan Bateman
Alan.Bateman at oracle.com
Tue Mar 7 12:11:41 UTC 2023
On 07/03/2023 08:21, Eirik Bjørsnøs wrote:
> :
>
> Seems we reached consensus that the right thing to do here is to add
> an `@apiNote`
>
> Here's a draft PR with a suggested CSR:
>
> https://github.com/openjdk/jdk/pull/12899
> <https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/12899__;!!ACWV5N9M2RV99hQ!JHKVw9-_twtqnwpDZOujHDa0DGd3b9cZnWu0dIC3RjeLReueGxSik19nbAJsNQg05qvwbGKnqEQXFfs$>
>
Yes, I think an apiNote is the best thing to do here.
For the note then you might want to change it to "ZipEntry e = ..."
because the reader see the trailing slash after dir so it is obviously a
directory. Also the javadoc uses "crc-32" rather than "crc" should we
should keep that consistent if we can.
-Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/core-libs-dev/attachments/20230307/ee1ecf51/attachment.htm>
More information about the core-libs-dev
mailing list