On Fri, 10 Jan 2025 07:30:04 GMT, Peter Levart <plev...@openjdk.org> wrote:

> [JDK-8169880](https://bugs.openjdk.org/browse/JDK-8169880) removed a 
> debugging option to disable caching of reflective data in java.lang.Class, 
> which had a consequence that now Class#reflectiveData method guarantees to 
> return a non-null result. Existing callers of that method were left intact 
> and still contain null checks and branches that are never taken after this 
> change. Some new callers were added later and they don't have null checks. I 
> propose a patch to remove these unneeded null checks and never taken branches.

Good; now no usage of `reflectionData()` involves redundant null checks. Also 
it's 2025 so you get the privilege to bump copyright year on `Class.java`.

-------------

Marked as reviewed by liach (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23025#pullrequestreview-2541769274

Reply via email to