RFR: 8347397: Cleanup of JDK-8169880 [v2]

Alan Bateman alanb at openjdk.org
Fri Jan 10 09:38:45 UTC 2025


On Fri, 10 Jan 2025 08:28:58 GMT, Peter Levart <plevart at 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.
>
> Peter Levart has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8347397: Happy New Year 2025

Marked as reviewed by alanb (Reviewer).

Welcome back, this looks fine.

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

PR Review: https://git.openjdk.org/jdk/pull/23025#pullrequestreview-2541958602
PR Comment: https://git.openjdk.org/jdk/pull/23025#issuecomment-2582189843


More information about the core-libs-dev mailing list