RFR: 8371953: Document null handling in core reflection APIs [v2]
Chen Liang
liach at openjdk.org
Sun Nov 16 15:45:24 UTC 2025
> A lot of core reflection APIs are from antique times, which have their own null handling behavior. Such behaviors are often not documented in the specification; we should document rejected null arguments and accepted null arguments (including array elements) explicitly.
>
> In the investigation, I found `Class.isNestmateOf` (inconsistent) and `AnnotatedType`'s implementation of `AnnotatedElement` methods (required by specification) are missing null checks. I consider these unlikely to be a user dependency and added new null checks.
Chen Liang has updated the pull request incrementally with three additional commits since the last revision:
- Rephrase for parameterTypes contains null
- Rename tests to be more specific
- Split annotated type new checks to another patch
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/28336/files
- new: https://git.openjdk.org/jdk/pull/28336/files/c42f70fb..a93a84ff
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=28336&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=28336&range=00-01
Stats: 134 lines in 9 files changed: 4 ins; 103 del; 27 mod
Patch: https://git.openjdk.org/jdk/pull/28336.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/28336/head:pull/28336
PR: https://git.openjdk.org/jdk/pull/28336
More information about the core-libs-dev
mailing list