> 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: Alan's review - don't repeat NullPointerException specification within same class ------------- Changes: - all: https://git.openjdk.org/jdk/pull/17728/files - new: https://git.openjdk.org/jdk/pull/17728/files/f92c5726..59d4df6b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17728&range=02-03 Stats: 6 lines in 3 files changed: 0 ins; 6 del; 0 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