Looks good, Lance. nit: a couple of comments (line 128,133) don't end with periods.
Naoto On 8/6/20 3:32 PM, Lance Andersen wrote:
Hi all, Please review the webrev for: https://bugs.openjdk.java.net/browse/JDK-8251202 <https://bugs.openjdk.java.net/browse/JDK-8251202>, which adds missing javadoc comments to java.util.zip.ZipConstants. The webrev can be found at: http://cr.openjdk.java.net/~lancea/8251205/webrev.00/ The CSR, which has been approved, can be found at: https://bugs.openjdk.java.net/browse/JDK-8251206 <https://bugs.openjdk.java.net/browse/JDK-8251206> Best Lance ------------------ Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 [email protected]
