> 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: no need for code comment ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17728/files - new: https://git.openjdk.org/jdk/pull/17728/files/93bccb00..6a9a59c8 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=05 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=04-05 Stats: 22 lines in 7 files changed: 0 ins; 21 del; 1 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