On Wed, 7 Jun 2023 22:02:57 GMT, Mandy Chung <mch...@openjdk.org> wrote:

>> JEP 416 integrated in JDK 18 and since then, only a couple minor issues has 
>> been reported.  Those issues were related with exception being thrown with 
>> invalid arguments.   We propose to remove the old core reflection 
>> implementation in JDK 22.  The `-Djdk.reflect.useDirectMethodHandle=false` 
>> workaround to revert to the old implementation will stop to work.
>
> 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.

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

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

Reply via email to