> Can I please get a review of this doc-only change which updates the javadoc > of several classes in `java.util.jar` and `java.util.zip` to specify their > behaviour when `null` arguments are passed to the constructor or methods of > those classes? > > For these updated classes, I have individually checked that they indeed throw > a `NullPointerException` when `null` is passed to their constructor or > methods. The couple of places where `null` is accepted have been updated to > mention that `null` is allowed.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: include CheckedInputStream and CheckedOutputStream ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17728/files - new: https://git.openjdk.org/jdk/pull/17728/files/298d0a0a..e4c724fc Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=00-01 Stats: 13 lines in 2 files changed: 7 ins; 0 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/17728.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/17728/head:pull/17728 PR: https://git.openjdk.org/jdk/pull/17728