> 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: make <p> usage consistent with other similar usages in the file ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17728/files - new: https://git.openjdk.org/jdk/pull/17728/files/59d4df6b..93bccb00 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=04 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=03-04 Stats: 7 lines in 3 files changed: 4 ins; 0 del; 3 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
