> Please review this PR which updates the specification of > `ZipOutputStream.setComment(String)` to match the long-standing behavior of > throwing `IllegalArgumentException` when a comment contains characters which > are unmappable using the `Charset` passed in the constructor. > > A new tests is added to reproduce calling setComment with unmappable > characters and verify that it throws IAE. > > A CSR has been drafted. Its specification section will be updated after an > initial round of review of the specification text in this PR.
Eirik Bjørsnøs has updated the pull request incrementally with one additional commit since the last revision: Add null / empty string note to ZipEntry::setComment. Use "an empty string" consistently. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/30338/files - new: https://git.openjdk.org/jdk/pull/30338/files/a891743b..7ae703a9 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=30338&range=06 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30338&range=05-06 Stats: 4 lines in 2 files changed: 1 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/30338.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/30338/head:pull/30338 PR: https://git.openjdk.org/jdk/pull/30338
