> 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
