On Fri, 10 Jan 2025 08:28:58 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.
>
> 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

Reply via email to