RFR: 8305104: Remove the old core reflection implementation [v2]

Mandy Chung mchung at openjdk.org
Thu Jun 8 16:47:00 UTC 2023


On Thu, 8 Jun 2023 08:42:41 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Mandy Chung has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fix merge issue
>
> src/java.base/share/classes/jdk/internal/reflect/ReflectionFactory.java line 578:
> 
>> 576:     // then switch to the bytecode-based implementations.
>> 577: 
>> 578:     private static final Config DEFAULT_CONFIG = new Config(false, // useNativeAccessorOnly
> 
> The block comment just before this will need updating.  There's another one in Config.config where it uses the default/native implementation in early startup.

Thanks for pointing this out.  Updated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14371#discussion_r1223305554


More information about the hotspot-dev mailing list